openedx / edx-platform

The Open edX LMS & Studio, powering education sites around the world!
https://openedx.org
GNU Affero General Public License v3.0
7.09k stars 3.79k forks source link

fix: let the platform decide to how login the users #34995

Open CodeWithEmad opened 3 weeks ago

CodeWithEmad commented 3 weeks ago

We force users to go to authentication mfe for login/registration and prevent login behavior tweaks like having THIRD_PARTY_AUTH_HINT in your site configuration which redirects users to a TPA when they visit /login.

Close #34990

openedx-webhooks commented 3 weeks ago

Thanks for the pull request, @CodeWithEmad! Please note that it may take us up to several weeks or months to complete a review and merge your PR.

Feel free to add as much of the following information to the ticket as you can:

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.

Please let us know once your PR is ready for our review and all tests are green.

CodeWithEmad commented 2 weeks ago

Any idea? Error: Failed to CreateArtifact: Received non-retryable error: Failed request: (409) Conflict: an artifact with this name already exists on the workflow run

mphilbrick211 commented 2 weeks ago

Any idea? Error: Failed to CreateArtifact: Received non-retryable error: Failed request: (409) Conflict: an artifact with this name already exists on the workflow run

Hi @CodeWithEmad! If needed, I can see if someone on the Axim team can help with this.

CodeWithEmad commented 2 weeks ago

@mphilbrick211 That would be amazing. Thank you.