-
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
Fix exitTo link by adding missing ? #5135
Conversation
Trying to figure out QA steps |
|
Fix exitTo link by adding missing ? (cherry picked from commit 81ff8d4)
🚀 Cherry-picked to staging by @ctkochan22 in version: 1.0.94-1 🚀
@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes. |
🚀 Deployed to production by @yuwenmemon in version: 1.0.94-1 🚀
|
🚀 Deployed to staging by @ctkochan22 in version: 1.0.94-2 🚀
|
@ctkochan22 @roryabraham Can you help us and provide QA steps (if any) for this PR |
Its a bit tricky. I tested it on staging already though |
@mvtglobally you can check off! |
🚀 Deployed to production by @yuwenmemon in version: 1.0.95-1 🚀
|
@roryabraham or @tgolen
Fix link by adding missing
?
Details
Fixed Issues
$ #5105
Original PR: https://github.com/Expensify/App/pull/4789/files#diff-fed84caebc7e71b17fe9ac17cd04c5ec5aa8ce8a9f2ddc2ad33eb77253560a2eL91
Tests
Strangely annoying. Need to setup a workspace where you have the cards enabled so that you can click "manage cards"
QA Steps
Tested On
Screenshots
Web
Mobile Web
Desktop
iOS
Android