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

Frontend Header #24

Closed
1 task
ghassanmas opened this issue Oct 19, 2021 · 2 comments
Closed
1 task

Frontend Header #24

ghassanmas opened this issue Oct 19, 2021 · 2 comments

Comments

@ghassanmas
Copy link
Member

Following our meeting on #21 (19-OCT-21).

This issue is intended to be as a reference and for discussion regarding the header component, it's intended to be used as a reference and to keep track of the logs todos..etc.

Background:

The learning MFE is currently is using its own hard-coded version of the headers because (maybe?) the learning app has a specific use case where the current frontend app component doesn't meet. While discussing this today, numerous suggestions/ideas were discussed (not decisions), for example, we could rebuild the component in a way where it would be customizable, extensible...etc

That being said before committing to how or what exactly should be done to resolve it, it would make sense to look over the cases in which how the MFE apps are using the headers.

To Do:

  • Document use cases of the headers in the MFE (in order to help create the path of the decision process of how the component should be an article).

Refs

  • Frontend app compoenet repo
  • Tne learning MFE repo
  • Discussion about this issue at Mitx issue
@davidjoy
Copy link
Contributor

davidjoy commented Nov 9, 2021

I'm inclined to close out this issue in favor of openedx/wg-build-test-release#81 where all the work seems to be happening.

@ghassanmas
Copy link
Member Author

@davidjoy totally agree

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants