-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
chore: fixed list shuffles on RandomAppList #1163
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@developerfred thanks for taking a swing at this!
Looking at the preview deploy however, the issue persists:
As mentioned in the issue, please make sure it works upon building the app, not just in your local environment. Thanks!
@samajammin I believe I discovered the problem, we need to use hooks. Every time the component is shuffled the Intl loads. Putting the component to be the last one to load using userEffects. makes the component work normally. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@developerfred thanks for the quick fix! LGTM.
Nice! Thanks @samajammin |
@all-contributors please add @developerfred for code. |
I've put up a pull request to add @developerfred! 🎉 |
Description
Changing the logic of random numbers for correction or random reproduction
#1040
Related Issue
Screenshots (if appropriate):
https://share.getcloudapp.com/2Nu5prrO