-
Notifications
You must be signed in to change notification settings - Fork 22
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 that submit draft keeps on spinning #951
Conversation
retry checking for the latest proposal if it matches title and description
@race-of-sloths include |
@petersalomonsen Thank you for your contribution! Your pull request is now a part of the Race of Sloths! Current status: waiting for finalizationThe pull request is merged, you have 24 hours to finalize your scoring. The scoring ends Tue Oct 8 19:40:46 2024
Your contribution is much appreciated with a final score of 3! What is the Race of SlothsRace of Sloths is a friendly competition where you can participate in challenges and compete with other open-source contributors within your normal workflow For contributors:
For maintainers:
Feel free to check our website for additional details! Bot commands
|
Love this! |
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.
LGTM
I think we need to make this change to RFP as well in infra.
@race-of-sloths score 3 |
@petersalomonsen Thank you for your contribution! Your pull request is now a part of the Race of Sloths! Current status: executed
Your contribution is much appreciated with a final score of 3! Another weekly streak completed, well done @petersalomonsen! To keep your weekly streak and get another bonus make pull request next week! Looking forward to see you in race-of-sloths What is the Race of SlothsRace of Sloths is a friendly competition where you can participate in challenges and compete with other open-source contributors within your normal workflow For contributors:
For maintainers:
Feel free to check our website for additional details! Bot commands
|
Based on the report that some user only saw the submit button spinner keeping on spinning, which I assume is as reproduced in the test that can be seen in the video here:
create-proposal-spinner-without-fix.mp4
The test reproduces a scenario where the transaction completes, but the view call for getting all proposal ids does not show the latest proposal immediately, which might happen in the case the transaction is not fully finalized. The previous implementation will then just keep on spinning, and so it is assumed that this is what the user experienced.
By retrying checking for the latest proposal until it matches the submitted title and description, the submitted proposal will be found eventually, and the submit button will stop spinning, and the new proposal should display.
With the fix, the submit button stops spinning, and the submitted proposal is displayed:
create-proposal-spinner-fix.mp4