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

[NO QA] Improve workspace welcome note #5714

Merged
merged 2 commits into from
Oct 7, 2021

Conversation

Beamanator
Copy link
Contributor

@Beamanator Beamanator commented Oct 7, 2021

Details

We default to adding "workspace" to the end of a new workspace name, which makes the welcome message read "You have been invited to (someone)'s workspace workspace..."

Fixed Issues

$ https://github.com/Expensify/Expensify/issues/179319

Tests

  1. Sign in to an account w/ a workspace
  2. Share the workspace
  3. Run bwm to send the shared workspace email notification
  4. Verify the message text is updated (try in spanish too, if you're so inclined)

QA Steps

Not needed, just a small copy change

Tested On

  • Web
  • Mobile Web
  • Desktop
  • iOS
  • Android

Screenshots

Web

Note: only the bottom circled text is fixed in this PR, the email subject will be fixed in Web-E

Screen Shot 2021-10-07 at 11 13 59 AM

Mobile Web

Desktop

iOS

Android

@Beamanator Beamanator added the Internal Requires API changes or must be handled by Expensify staff label Oct 7, 2021
@Beamanator Beamanator requested a review from a team as a code owner October 7, 2021 18:11
@Beamanator Beamanator self-assigned this Oct 7, 2021
@MelvinBot MelvinBot requested review from TomatoToaster and removed request for a team October 7, 2021 18:11
@Beamanator
Copy link
Contributor Author

Ready for review 👍 (added the spanish translation late, oops!)

@TomatoToaster TomatoToaster merged commit a1640d8 into main Oct 7, 2021
@TomatoToaster TomatoToaster deleted the beaman-improveWorkspaceWelcomeNote branch October 7, 2021 20:16
@OSBotify
Copy link
Contributor

OSBotify commented Oct 7, 2021

✋ 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 Oct 7, 2021

🚀 Deployed to staging by @TomatoToaster in version: 1.1.7-0 🚀

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

@OSBotify
Copy link
Contributor

🚀 Deployed to production by @AndrewGable in version: 1.1.7-24 🚀

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
Internal Requires API changes or must be handled by Expensify staff
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants