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

Correctly exclude system emails from inviting in workspaces #5951

Merged
merged 5 commits into from
Oct 22, 2021

Conversation

parasharrajat
Copy link
Member

Details

Fixed Issues

Follow up ##4775

@parasharrajat parasharrajat requested a review from a team as a code owner October 19, 2021 18:18
@MelvinBot MelvinBot requested review from Julesssss and removed request for a team October 19, 2021 18:18
Julesssss
Julesssss previously approved these changes Oct 20, 2021
@Julesssss
Copy link
Contributor

FYI @parasharrajat, there's a simple import conflict to resolve.

@parasharrajat
Copy link
Member Author

@Julesssss Ready.

@parasharrajat
Copy link
Member Author

Ready for review

@marcaaron
Copy link
Contributor

Restarted the e2e test to see if it will pass

@parasharrajat
Copy link
Member Author

No luck. Gems installation is failing.

@parasharrajat
Copy link
Member Author

parasharrajat commented Oct 22, 2021

Please, Can this be merged if everything is good here?

@marcaaron
Copy link
Contributor

e2e test is failing do we know why?

@parasharrajat
Copy link
Member Author

Yeah. There was some config issue which is fixed and I have merged main so they will pass shortly.

@parasharrajat
Copy link
Member Author

@marcaaron Ready

@marcaaron marcaaron merged commit 6d42627 into Expensify:main Oct 22, 2021
@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

🚀 Deployed to staging by @marcaaron in version: 1.1.8-10 🚀

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

@mvtglobally
Copy link

@parasharrajat @marcaaron Do we need to QA this PR? Can you please share steps

@parasharrajat
Copy link
Member Author

No QA. thanks

@OSBotify
Copy link
Contributor

🚀 Deployed to production by @roryabraham in version: 1.1.10-2 🚀

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.

None yet

5 participants