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

Update onyx & Fix the enable-payments page #4604

Merged
merged 2 commits into from
Aug 13, 2021

Conversation

parasharrajat
Copy link
Member

Details

Fixed Issues

$ #4500
Fixes #4392

Tests | QA Steps

  1. Navigate to staging.new.expensify.com/enable-payments.
  2. Refresh the page

Tested On

  • Web
  • Mobile Web

Screenshots

Web

enable-payments.mp4

@parasharrajat parasharrajat requested a review from a team as a code owner August 12, 2021 12:44
@MelvinBot MelvinBot requested review from AndrewGable and removed request for a team August 12, 2021 12:44
@parasharrajat
Copy link
Member Author

This is straightforward PR which simply upgrade Onyx version.

@AndrewGable
Copy link
Contributor

Looks good @parasharrajat - I am ready to merge when you fix merge conflicts

@AndrewGable
Copy link
Contributor

🤔 - Now there are no changes

@parasharrajat
Copy link
Member Author

@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.

@AndrewGable
Copy link
Contributor

Can we confirm the fix is live first? Either on staging or locally?

@parasharrajat
Copy link
Member Author

Ok, this is the PR #4610 which upgrades the Onyx. Looks like yet to be pushed to staging.

@parasharrajat
Copy link
Member Author

Maybe it's better to merge so that QA can be done separately.

@AndrewGable
Copy link
Contributor

Good point 👍

@AndrewGable AndrewGable merged commit 1c3a797 into Expensify:main Aug 13, 2021
@OSBotify
Copy link
Contributor

✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

@OSBotify
Copy link
Contributor

🚀 Deployed to staging by @AndrewGable in version: 1.0.85-10 🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅

@marcaaron marcaaron added the DeployBlockerCash This issue or pull request should block deployment label Aug 17, 2021
@isagoico
Copy link

@marcaaron I guess the Deploy blocker label was added because of the blank page when navigating to enable-payments
Should I create a separate issue or is this already being addressed?
image

@marcaaron
Copy link
Contributor

I don't think there is any need for a new issue.

@marcaaron
Copy link
Contributor

i.e. both issues should be resolved once we fix the new blocker.

@parasharrajat
Copy link
Member Author

I don't think that I have to do something here. Revert PR will fix the original issue as well.

@isagoico
Copy link

@marcaaron This issue is still reproducible in 1.0.86-2
image

@parasharrajat
Copy link
Member Author

@isagoico the issue you are reporting now has a different cause than what we fixed.

@isagoico
Copy link

oh well I'll go ahead and create an issue for this one. Thanks Rajat!

@marcaaron
Copy link
Contributor

There's no spinner anymore so it seems maybe a new issue? Not totally sure.

@isagoico
Copy link

Reported the issue here #4719

@botify
Copy link

botify commented Aug 25, 2021

This has been deployed to production and is now subject to a 7-day regression period.
If no regressions arise, payment will be issued on 2021-09-01. 🎊

@OSBotify
Copy link
Contributor

🚀 Deployed to production by @roryabraham in version: 1.0.86-11 🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 failure ❌
🕸 web 🕸 success ✅

@botify
Copy link

botify commented Aug 30, 2021

This has been deployed to production and is now subject to a 7-day regression period.
If no regressions arise, payment will be issued on 2021-09-06. 🎊

@MitchExpensify
Copy link
Contributor

No regressions mentioned here, paying in Upwork!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DeployBlockerCash This issue or pull request should block deployment
Projects
None yet
7 participants