-
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: Remaining pages for workspace #6557
Conversation
@Gonals Please review... |
@thienlnam I don't know why E2E test failed but we should be good to merge here. |
Hmm, are E2E tests failing on other PRs? If not then I don't think we should merge this. Going to re-run the test to see if it was just a flakey test |
We are good.. @thienlnam |
✋ 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 @thienlnam in version: 1.1.17-8 🚀
|
@parasharrajat Do we need to do any additional QA for this PR on top of QA for #6543? |
I think this PR is possible breaking the VBA flow -> #6620 Did a |
🚀 Deployed to production by @roryabraham in version: 1.1.18-3 🚀
|
Details
Follow up #6543
Fixed Issues
$ #6496
Tests | QA Steps
Same as preceding PR.
Tested On
Screenshots
Web
Mobile Web
Desktop
iOS
Android
output_file.mp4