Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@mahee96
Copy link
Member

@mahee96 mahee96 commented Nov 10, 2024

Issue #723:

  • Error when refreshing one of the apps sets the error for group context thereby creating same error signature for all.
  • This is misleading since the other apps could have proceeded with refresh but due to this "All or nothing" those refresh requests are also ignored BUT MARKED AS SAME ERROR.

Fix (Changes):

  • set error only to current AppOperationContext and not the group.context.
  • This will enable other refreshable apps to proceed without being interrupted.

NOTE: Since SideStore intentionally takes install path even for refresh request, the removeAppExtensions operation still requires UI context, this bug will be fixed in next PR.

@nythepegasus nythepegasus merged commit 489843f into SideStore:develop Nov 10, 2024
1 check passed
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