Skip to content

Adding the minimum version and the latest tested version of NVDA for add-on#16053

Merged
seanbudd merged 11 commits into
nvaccess:masterfrom
Nael-Sayegh:display-compatibility-extension
Jan 23, 2024
Merged

Adding the minimum version and the latest tested version of NVDA for add-on#16053
seanbudd merged 11 commits into
nvaccess:masterfrom
Nael-Sayegh:display-compatibility-extension

Conversation

@Nael-Sayegh

@Nael-Sayegh Nael-Sayegh commented Jan 16, 2024

Copy link
Copy Markdown
Contributor

Link to issue number:

closes #15776

Summary of the issue:

Having the ability to see the minimum version and the latest tested version of NVDA for an extension in the addonStore.

Description of user facing changes

In the addonStore, the user will be able to see the minimum version and the latest tested version of NVDA in the details of an extension.

Description of development approach

Added in the source file\gui\addonStoreGui\controls\details.py in the refresh method, I added compatibility information under the installed version of the module.

Testing strategy:

Go to the addonStore, then select an installed extension, go to the "Other Information" editing area and read the section to see the minimum version and latest tested version.

Known issues with pull request:

Code Review Checklist:

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

Comment thread source/gui/addonStoreGui/controls/details.py Outdated
Comment thread source/gui/addonStoreGui/controls/details.py Outdated

@CyrilleB79 CyrilleB79 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

A rephrasing of the change log.

And also, could you change the title of this PR for clarity, replacing "extension" by "add-on"? "add-on" is the word used in NVDA's English version.

Comment thread user_docs/en/changes.t2t Outdated
@Nael-Sayegh Nael-Sayegh force-pushed the display-compatibility-extension branch from ac4151b to 4737c89 Compare January 17, 2024 09:40
@Nael-Sayegh Nael-Sayegh changed the title Adding the minimum version and the latest tested version of NVDA for extension Adding the minimum version and the latest tested version of NVDA for add-on Jan 17, 2024
@Nael-Sayegh

Copy link
Copy Markdown
Contributor Author

@CyrilleB79 Thank you for your advice. I have made the changes, I think it is correct now.

@Nael-Sayegh Nael-Sayegh marked this pull request as ready for review January 17, 2024 13:28
@Nael-Sayegh Nael-Sayegh requested a review from a team as a code owner January 17, 2024 13:28
@Nael-Sayegh Nael-Sayegh requested a review from seanbudd January 17, 2024 13:28
Comment thread user_docs/en/changes.t2t
Comment thread user_docs/en/changes.t2t Outdated
Comment thread source/gui/addonStoreGui/controls/details.py Outdated
Comment thread source/gui/addonStoreGui/controls/details.py Outdated
Comment thread user_docs/en/changes.t2t Outdated
@seanbudd seanbudd marked this pull request as draft January 19, 2024 00:34
@Nael-Sayegh

Copy link
Copy Markdown
Contributor Author

@seanbudd I have moved the description in "changes" section and add section add-on store in "changes".
I have done also your prosition of modifications.

@Nael-Sayegh Nael-Sayegh marked this pull request as ready for review January 19, 2024 09:53
Comment thread user_docs/en/changes.t2t
Comment thread user_docs/en/changes.t2t Outdated
Comment thread user_docs/en/changes.t2t
@seanbudd seanbudd marked this pull request as draft January 22, 2024 01:02
Co-authored-by: Sean Budd <seanbudd123@gmail.com>
Comment thread user_docs/en/changes.t2t Outdated
Nael-Sayegh and others added 2 commits January 22, 2024 09:17
Co-authored-by: Cyrille Bougot <cyrille.bougot2@laposte.net>
@Nael-Sayegh Nael-Sayegh marked this pull request as ready for review January 22, 2024 10:23
@seanbudd seanbudd added the conceptApproved Similar 'triaged' for issues, PR accepted in theory, implementation needs review. label Jan 22, 2024
Comment thread user_docs/en/changes.t2t Outdated
Comment thread user_docs/en/changes.t2t
@seanbudd seanbudd marked this pull request as draft January 23, 2024 00:17
Nael-Sayegh and others added 2 commits January 23, 2024 08:12
Co-authored-by: Sean Budd <seanbudd123@gmail.com>
@Nael-Sayegh Nael-Sayegh marked this pull request as ready for review January 23, 2024 12:29

@seanbudd seanbudd left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks @Nael-Sayegh

@seanbudd seanbudd merged commit 5b4ecef into nvaccess:master Jan 23, 2024
@nvaccessAuto nvaccessAuto added this to the 2024.2 milestone Jan 23, 2024
@Nael-Sayegh Nael-Sayegh deleted the display-compatibility-extension branch January 26, 2024 08:47
@Nael-Sayegh Nael-Sayegh restored the display-compatibility-extension branch January 26, 2024 08:47
@Nael-Sayegh Nael-Sayegh deleted the display-compatibility-extension branch January 26, 2024 08:48
Nael-Sayegh added a commit to Nael-Sayegh/nvda that referenced this pull request Feb 15, 2024
…add-on (nvaccess#16053)

closes nvaccess#15776

Summary of the issue:
Having the ability to see the minimum version and the latest tested version of NVDA for an extension in the addonStore.

Description of user facing changes
In the addonStore, the user will be able to see the minimum version and the latest tested version of NVDA in the details of an extension.

Description of development approach
Added in the source file\gui\addonStoreGui\controls\details.py in the refresh method, I added compatibility information under the installed version of the module.
SaschaCowley pushed a commit to SaschaCowley/nvda that referenced this pull request Feb 27, 2024
…add-on (nvaccess#16053)

closes nvaccess#15776

Summary of the issue:
Having the ability to see the minimum version and the latest tested version of NVDA for an extension in the addonStore.

Description of user facing changes
In the addonStore, the user will be able to see the minimum version and the latest tested version of NVDA in the details of an extension.

Description of development approach
Added in the source file\gui\addonStoreGui\controls\details.py in the refresh method, I added compatibility information under the installed version of the module.
Adriani90 pushed a commit to Adriani90/nvda that referenced this pull request Mar 13, 2024
…add-on (nvaccess#16053)

closes nvaccess#15776

Summary of the issue:
Having the ability to see the minimum version and the latest tested version of NVDA for an extension in the addonStore.

Description of user facing changes
In the addonStore, the user will be able to see the minimum version and the latest tested version of NVDA in the details of an extension.

Description of development approach
Added in the source file\gui\addonStoreGui\controls\details.py in the refresh method, I added compatibility information under the installed version of the module.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

conceptApproved Similar 'triaged' for issues, PR accepted in theory, implementation needs review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Ability to show the minimum and last tested NVDA versions of an add-on in the descriptions of the add on store.

4 participants