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

Add-on store: if the add-on download fails, you should be able to re-download the add-on #17090

Closed
hwf1324 opened this issue Aug 31, 2024 · 3 comments · Fixed by #17104
Closed
Labels
feature/addon-store Features / behavior of the add-on Store feature p5 https://github.com/nvaccess/nvda/blob/master/projectDocs/issues/triage.md#priority triaged Has been triaged, issue is waiting for implementation.
Milestone

Comments

@hwf1324
Copy link
Contributor

hwf1324 commented Aug 31, 2024

Is your feature request related to a problem? Please describe.

When the add-on fails to download, it could simply be the network, and when I configure the network and want to retry the download, the fastest way to do it would be to switch to the other tab on the way back, which is cumbersome and not intuitive.

Describe the solution you'd like

Add “Retry”/“Redownload” Action when the add-on fails to download.

Describe alternatives you've considered

Additional context

@seanbudd seanbudd changed the title Add-on store: if the add-on download fails, you can re-download the add-on Add-on store: if the add-on download fails, you should be able to re-download the add-on Sep 2, 2024
@seanbudd
Copy link
Member

seanbudd commented Sep 2, 2024

I think the install action should just be allowed in this case, rather than making a new action

@seanbudd seanbudd added p5 https://github.com/nvaccess/nvda/blob/master/projectDocs/issues/triage.md#priority feature triaged Has been triaged, issue is waiting for implementation. feature/addon-store Features / behavior of the add-on Store labels Sep 2, 2024
@hwf1324
Copy link
Contributor Author

hwf1324 commented Sep 3, 2024

I think the install action should just be allowed in this case, rather than making a new action

That's great, but it seems a bit difficult to realise.

My general idea is to re-determine the available Actions via getStatus when fetching the available Actions when the status of the add-on is Download Failed.

But it seems that I have to add some tedious code if I want to get the external context in the abstract model.

@seanbudd
Copy link
Member

seanbudd commented Sep 3, 2024

@hwf1324 - I think this patch should cover the issue

AddonActionVM(
  # Translators: Label for an action that retries the selected addon
  displayName=pgettext("addonStore", "&Retry"),
  actionHandler=self.getAddon,
  validCheck=lambda aVM: aVM.status in {AvailableAddonStatus.DOWNLOAD_FAILED, AvailableAddonStatus.INSTALL_FAILED},
  actionTarget=selectedListItem,
),

seanbudd pushed a commit that referenced this issue Sep 9, 2024
Closed #17090

Summary of the issue:
There is no Action to retry the installation when downloading/installing an add-on via the Add-on Store fails.

Description of user facing changes
When downloading/installing an add-on fails, you can now retry the installation

Description of development approach
Add Retry Action for failed download/installation status.
@github-actions github-actions bot added this to the 2025.1 milestone Sep 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature/addon-store Features / behavior of the add-on Store feature p5 https://github.com/nvaccess/nvda/blob/master/projectDocs/issues/triage.md#priority triaged Has been triaged, issue is waiting for implementation.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants