Skip to content

Add-on store: Fix handling of external add-ons#15223

Merged
seanbudd merged 1 commit into
betafrom
fix-15218
Jul 31, 2023
Merged

Add-on store: Fix handling of external add-ons#15223
seanbudd merged 1 commit into
betafrom
fix-15218

Conversation

@seanbudd

@seanbudd seanbudd commented Jul 31, 2023

Copy link
Copy Markdown
Member

Link to issue number:

Fixes #15218

Summary of the issue:

Add-ons which were being installed from an external source were treated like an installed add-on.
This resulted in trying to get manifest information for the installed add-ons collection, rather than the bundle that the install is coming from.
When trying to install an incompatible add-on from an external source, an error is raised, as manifest data is not fetched correctly.
This prevents the confirmation dialog from appearing and prevents the user from installing incompatible add-ons from external sources.

Description of user facing changes

Add-ons which are incompatible can be installed from an external source.

Description of development approach

Instead, use the manifest data from the external add-on bundle.

Testing strategy:

Manually test STR in #15218
Smoke test installing add-ons using the [manual test plan] (https://github.com/nvaccess/nvda/blob/master/tests/manual/addonStore.md)

Known issues with pull request:

None

Change log entries:

N/A

Code Review Checklist:

  • Pull Request description:
    • description is up to date
    • change log entries
  • Testing:
    • Unit tests
    • System (end to end) tests
    • Manual testing
  • API is compatible with existing add-ons.
  • Documentation:
    • User Documentation
    • Developer / Technical Documentation
    • Context sensitive help for GUI changes
  • UX of all users considered:
    • Speech
    • Braille
    • Low Vision
    • Different web browsers
    • Localization in other languages / culture than English
  • Security precautions taken.

@seanbudd seanbudd requested a review from a team as a code owner July 31, 2023 01:18
@seanbudd seanbudd requested review from michaelDCurran and removed request for a team July 31, 2023 01:18
@seanbudd seanbudd added this to the 2023.2 milestone Jul 31, 2023
@seanbudd seanbudd merged commit c788619 into beta Jul 31, 2023
@seanbudd seanbudd deleted the fix-15218 branch July 31, 2023 03:24
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.

2 participants