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: excludes "ambitiouspilots-toolbar-pushback" from the Incompatible Add-ons #485

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

batata7elwe
Copy link

Removed "ambitiouspilots-toolbar-pushback" from the Incompatible Add-ons list.

The "ambitiouspilots-toolbar-pushback" add-on will no longer trigger the "Incompatible Add-ons!" pop-up in the installer when updating the A32NX.

Fixes #[issue_no]

Summary of Changes

Removed "ambitiouspilots-toolbar-pushback" from the Incompatible Add-ons list.

Screenshots (if necessary)

Additional context

The toolbar pushback add-on still triggered the "Incompatible Add-ons!" pop-up in the installer when updating the A32NX, even though the issues caused by this add-on have been fixed a long time ago (according to some of the developers).

Discord username (if different from GitHub): batata7elwe

Removed "ambitiouspilots-toolbar-pushback" from the Incompatible Add-ons list.
@batata7elwe batata7elwe marked this pull request as ready for review October 30, 2024 03:08
@JasonO99
Copy link

How confident are we this addon does not cause issues anymore? It seems to work most of the time, but I don’t think it’s perfect.

Someone just today resolved an issue they had by not using the tool on the Discord; https://discord.com/channels/738864299392630914/785976111875751956/1301091398430232617

There is always a chance of coincidence but I suppose It depends what threshold of issues vs flights without issue there is - but we won’t know the latter.

@tracernz
Copy link
Member

tracernz commented Oct 30, 2024

It’s certainly not in the same class as other addons that outright override our files. If we apply a consistent criteria then I think things like GSX would have to be added alongside this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants