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(mvux): Fix possible cross threading exception in refresh command #2448

Merged
merged 2 commits into from
Jul 19, 2024

Conversation

dr1rrb
Copy link
Member

@dr1rrb dr1rrb commented Jul 19, 2024

closes #2444

Bugfix

Fix possible cross threading exception in refresh command

What is the current behavior?

Refresh command may remain CanExecute = false after a first execution

What is the new behavior?

🙃

PR Checklist

Please check if your PR fulfills the following requirements:

  • Tested code with current supported SDKs
  • Docs have been added/updated which fit documentation template. (for bug fixes / features)
  • Unit Tests and/or UI Tests for the changes have been added (for bug fixes / features) (if applicable)
  • Wasm UI Tests are not showing unexpected any differences. Validate PR Screenshots Compare Test Run results.
  • Contains NO breaking changes
  • Updated the Release Notes
  • Associated with an issue (GitHub or internal)

Other information

Internal Issue (If applicable):

@jeromelaban
Copy link
Member

@Mergifyio backport release/stable/4.2

Copy link

mergify bot commented Jul 19, 2024

backport release/stable/4.2

✅ Backports have been created

@dr1rrb dr1rrb merged commit 066df73 into main Jul 19, 2024
16 of 17 checks passed
@dr1rrb dr1rrb deleted the dev/dr/refThreading branch July 19, 2024 13:18
jeromelaban added a commit that referenced this pull request Aug 2, 2024
….2/pr-2448

fix(mvux): Fix possible cross threading exception in refresh command (backport #2448)
@modula2019
Copy link

@dr1rrb Good day to you and your team. I am still having this issue I reported here. https://github.com/unoplatform/uno.extensions/issues/2444
I thought the bug was fixed in release 4.2. The bug persists with Desktop platform using uno.sdk 5.3.90. The repro is attached here again.
UnoApp1.zip

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.

BUG On Refreshing FeedView- MVUX (Desktop Platform)
3 participants