-
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
[No QA] Revert "Toggle staging and production server api" #10920
Conversation
|
PR Reviewer Checklist
|
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.
Thanks for the quick fix
@thienlnam looks like this was merged without passing tests. Please add a note explaining why this was done and remove the |
Contributor checklist test was failing, just merged it regardless |
…ng-server (cherry picked from commit dbe50c1)
…-10920 🍒 Cherry pick PR #10920 to staging 🍒
Tested logging in staging after the revert and worked good. |
🚀 Cherry-picked to staging by @thienlnam in version: 1.1.99-2 🚀
|
🚀 Cherry-picked to staging by https://github.com/AndrewGable in version: 1.3.28-2 🚀
|
The changes to allow us to use the Staging Web-E server are not working because of CSP.
We will need to update the rules so the requests go through just fine.
Reverts #10690
Fixes #10917