-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Update onyx & Fix the enable-payments
page
#4604
Conversation
This is straightforward PR which simply upgrade Onyx version. |
Looks good @parasharrajat - I am ready to merge when you fix merge conflicts |
🤔 - Now there are no changes |
@AndrewGable Looks like someone already updated the version. I don't think we have to merge this PR. There is nothing to merge and you can simply mark the issues completed to just notify that those are done. |
Can we confirm the fix is live first? Either on staging or locally? |
Ok, this is the PR #4610 which upgrades the Onyx. Looks like yet to be pushed to staging. |
Maybe it's better to merge so that QA can be done separately. |
Good point 👍 |
✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release. |
🚀 Deployed to staging by @AndrewGable in version: 1.0.85-10 🚀
|
@marcaaron I guess the Deploy blocker label was added because of the blank page when navigating to |
I don't think there is any need for a new issue. |
i.e. both issues should be resolved once we fix the new blocker. |
I don't think that I have to do something here. Revert PR will fix the original issue as well. |
@marcaaron This issue is still reproducible in 1.0.86-2 |
@isagoico the issue you are reporting now has a different cause than what we fixed. |
oh well I'll go ahead and create an issue for this one. Thanks Rajat! |
There's no spinner anymore so it seems maybe a new issue? Not totally sure. |
Reported the issue here #4719 |
This has been deployed to production and is now subject to a 7-day regression period. |
🚀 Deployed to production by @roryabraham in version: 1.0.86-11 🚀
|
This has been deployed to production and is now subject to a 7-day regression period. |
No regressions mentioned here, paying in Upwork! |
Details
Fixed Issues
$ #4500
Fixes #4392
Tests | QA Steps
Tested On
Screenshots
Web
enable-payments.mp4