Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix error popup in Arsenal Garage #1094

Merged
merged 2 commits into from
Mar 9, 2019
Merged

fix error popup in Arsenal Garage #1094

merged 2 commits into from
Mar 9, 2019

Conversation

commy2
Copy link
Contributor

@commy2 commy2 commented Mar 9, 2019

When merged this pull request will:

  • fixes error popup Picture logo_cba_ca.paa not found when opening the Virtual Garage
  • remove a version number from a place that will not get updated
  • attempt to hide the invisible target from the Arsenal using scopeArsenal = 0;

Predictably scopeArsenal = 0; doesn't work. It never works. I just leave it here as a reminder for everyone else that scopeArsenal indeed does not work.

https://i.imgur.com/C40ewkv.png

@commy2 commy2 added the Bug Fix label Mar 9, 2019
@commy2 commy2 added this to the 3.10 milestone Mar 9, 2019
@@ -2,15 +2,13 @@ class CfgMods {
class PREFIX {
author = "$STR_CBA_Author";
dir = "@CBA_A3";
name = "Community Base Addons v3.9.1";
Copy link
Contributor

@PabstMirror PabstMirror Mar 9, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

change it to 0.0.0
there is code to change on build
https://github.com/CBATeam/CBA_A3/blob/master/tools/make.py#L77
versionFiles = ["mod.cpp","addons\main_a3\CfgMods.hpp"]
https://github.com/CBATeam/CBA_A3/blob/master/tools/make.py#L597

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
name = "Community Base Addons v3.9.1";
name = "Community Base Addons v0.0.0";

Like this?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah, also need to change
https://github.com/CBATeam/CBA_A3/blob/master/tools/make.py#L77
versionFiles = ["mod.cpp","addons\main_a3\CfgMods.hpp"]

@commy2
Copy link
Contributor Author

commy2 commented Mar 9, 2019

@PabstMirror Merge?

picture = "x\cba\addons\main\logo_cba_ca.paa";
hidePicture = 1;
hideName = 1;
actionName = "Website";
action = "$STR_CBA_URL";
description = "Bugtracker: https://github.com/CBATeam/CBA_A3/issues<br/>Documentation: https://github.com/CBATeam/CBA_A3/wiki";
logo = "logo_cba_ca.paa";
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This works if you have the logo inside a PBO as well, not just root of @mod folder.

@commy2 commy2 merged commit c44e419 into master Mar 9, 2019
@commy2 commy2 deleted the arsenal-garage-fixes branch March 9, 2019 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants