-
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
Correctly exclude system emails from inviting in workspaces #5951
Conversation
FYI @parasharrajat, there's a simple import conflict to resolve. |
@Julesssss Ready. |
260f11f
to
90cdb86
Compare
Ready for review |
Restarted the e2e test to see if it will pass |
No luck. Gems installation is failing. |
Please, Can this be merged if everything is good here? |
e2e test is failing do we know why? |
Yeah. There was some config issue which is fixed and I have merged main so they will pass shortly. |
@marcaaron Ready |
✋ 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 @marcaaron in version: 1.1.8-10 🚀
|
@parasharrajat @marcaaron Do we need to QA this PR? Can you please share steps |
No QA. thanks |
🚀 Deployed to production by @roryabraham in version: 1.1.10-2 🚀
|
Details
Fixed Issues
Follow up ##4775