-
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: Added web link for desktop error page #7364
fix: Added web link for desktop error page #7364
Conversation
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.
cc: @NikkiWines
🎀 👀 🎀 C+ reviewed
Thanks for correcting the link. I missed that. |
Just a note that the linked QA steps don't work currently on dev - looks like someone fixed the room details issue. I would update the QA steps to direct the tester to go to a malformed URL like |
@NikkiWines Anyway to get this working for Desktop App? Because we can't enter URL there. |
@mananjadhav oh, right - in that case if we can use the following steps for desktop (this flow works fine on web, oddly):
Screen.Recording.2022-01-24.at.10.17.39.mov |
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.
👍
@mananjadhav since you mentioned tackling the url constant updates in a separate issue/PR, I'm going to go ahead and merge this - sound good? |
Yeah thanks, will you also help create an issue then? It'll be easier to track? |
@roryabraham created one already here |
✋ 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 @NikkiWines in version: 1.1.33-4 🚀
|
🚀 Deployed to production by @sketchydroide in version: 1.1.34-0 🚀
|
Details
Fixed Issues
$ #7316
Tests
QA Steps
Need to follow the reproduction steps from #6856 (comment) for Desktop.
Tested On
Screenshots
Web
Mobile Web
Desktop
iOS
Android