overhangio / tutor-mfe

This plugin makes it possible to easily add micro frontend (MFE) applications on top of an Open edX platform that runs with Tutor.
GNU Affero General Public License v3.0
22 stars 95 forks source link

Add INFO_EMAIL and PASSWORD_RESET_SUPPORT_LINK #141

Closed misilot closed 1 year ago

misilot commented 1 year ago

These two variables are used to generate links in the standard MFE Auth Site Template for resetting passwords.

Fixes #139

I don't currently have anyway to test with tutor-dev but I did apply this on our test instance running tutor local as a patch.

regisb commented 1 year ago

Right. I mentioned the dev environment because you will almost certainly have to implement the "openedx-lms-development-settings" patch, in addition to the production patch.

misilot commented 1 year ago

Thanks @regisb. I am willing to try and get it running.

What would be the most straightforward way to test my branch? I assume I will need to build tutor-mfe, but reading through the README in this repo, and the Tutor docs, I am not seeing if there is a way specify the github repo/branch as the source for the mfe.

Edit: Ohh, I think I should be able to install my version of the plugin locally via pip and build the mfe that way.

misilot commented 1 year ago

After changing to my tutor-mfe branch

image

After putting in a test email, the href is also populated (bottom left of image)

image

These changes did not require any additional patch to be applied to use the default CONTACT_EMAIL which is generated during launch.

misilot commented 1 year ago

@regisb done