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

fix: Added web link for desktop error page #7364

Merged
merged 3 commits into from
Jan 31, 2022

Conversation

mananjadhav
Copy link
Collaborator

Details

  • Added web link for desktop error page

Fixed Issues

$ #7316

Tests

  1. Tested link for the Desktop App
  • Verify that no errors appear in the JS console

QA Steps

Need to follow the reproduction steps from #6856 (comment) for Desktop.

  • Verify that no errors appear in the JS console

Tested On

  • Web
  • Mobile Web
  • Desktop
  • iOS
  • Android

Screenshots

Web

web-mobile-web-link

Mobile Web

Desktop

desktop-link

iOS

Android

@mananjadhav mananjadhav requested a review from a team as a code owner January 21, 2022 22:17
@MelvinBot MelvinBot requested review from NikkiWines and parasharrajat and removed request for a team January 21, 2022 22:17
parasharrajat
parasharrajat previously approved these changes Jan 21, 2022
Copy link
Member

@parasharrajat parasharrajat left a 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

@parasharrajat
Copy link
Member

Thanks for correcting the link. I missed that.

@NikkiWines
Copy link
Contributor

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 https://new.expensify.com/r/75431276____/details

@mananjadhav
Copy link
Collaborator Author

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 https://new.expensify.com/r/75431276____/details

@NikkiWines Anyway to get this working for Desktop App? Because we can't enter URL there.

@NikkiWines
Copy link
Contributor

@mananjadhav oh, right - in that case if we can use the following steps for desktop (this flow works fine on web, oddly):

  1. Create a new room using the + FAB
  2. Click on the room's icon to open up the room details
  3. Click on Settings
  4. Confirm you see the error page and that it directs you to switch to web
Screen.Recording.2022-01-24.at.10.17.39.mov

Copy link
Contributor

@NikkiWines NikkiWines left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@NikkiWines
Copy link
Contributor

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

@mananjadhav
Copy link
Collaborator Author

Yeah thanks, will you also help create an issue then? It'll be easier to track?

@NikkiWines
Copy link
Contributor

@roryabraham created one already here

@NikkiWines NikkiWines merged commit 7f0e7f8 into Expensify:main Jan 31, 2022
@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

OSBotify commented Feb 1, 2022

🚀 Deployed to staging by @NikkiWines in version: 1.1.33-4 🚀

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

@OSBotify
Copy link
Contributor

OSBotify commented Feb 2, 2022

🚀 Deployed to production by @sketchydroide in version: 1.1.34-0 🚀

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants