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

Add support for the authn MFE #90

Closed
3 tasks done
arbrandes opened this issue Nov 24, 2022 · 5 comments
Closed
3 tasks done

Add support for the authn MFE #90

arbrandes opened this issue Nov 24, 2022 · 5 comments
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@arbrandes
Copy link
Collaborator

arbrandes commented Nov 24, 2022

Upstream has implemented a fix to the branding issue. This might make it in as well!

@arbrandes
Copy link
Collaborator Author

We hit a brick wall for Olive: it seems the authentication API used by the Authn MFE requires edx-enterprise to be deployed.

Let's try again in Palm.

@arbrandes arbrandes modified the milestones: Olive.1, Palm.1 Dec 8, 2022
@arbrandes arbrandes removed their assignment Dec 8, 2022
@arbrandes arbrandes moved this from In progress to Backlog in Build-Test-Release Working Group Dec 8, 2022
@arbrandes arbrandes modified the milestones: Palm.1, Olive.1 Dec 9, 2022
@arbrandes arbrandes self-assigned this Dec 9, 2022
@arbrandes arbrandes changed the title Add support for frontend-app-authn feat: add support for the authn MFE Dec 9, 2022
@arbrandes arbrandes changed the title feat: add support for the authn MFE Add support for the authn MFE Dec 9, 2022
@arbrandes
Copy link
Collaborator Author

Turns out the presumed dependency was a red herring. We're good to include authn again, if we want to! I tested it, and it works.

@regisb
Copy link
Contributor

regisb commented Dec 9, 2022

It would be great if we could include all the new MFEs in the olive branch of the plugin, such that we leave some time for testers to at least have a look at the new MFEs. It's not much, but better than nothing.

@arbrandes
Copy link
Collaborator Author

@regisb, I've been rebasing tutor-mfe@olive on nightly, testing, then force-pushing as each is added, so that should be covered a few minutes after the nightly one is merged.

@arbrandes
Copy link
Collaborator Author

Aaaad we're in!

Repository owner moved this from Backlog to Done in Build-Test-Release Working Group Dec 9, 2022
Repository owner moved this from To Do - Prioritized for Current Sprint to Done in Axim Engineering Tasks Dec 9, 2022
Repository owner moved this from Backlog to In progress in Frontend Working Group Dec 9, 2022
@arbrandes arbrandes moved this from In progress to Closed in Frontend Working Group Dec 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants