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

feat: implement FooterSlot #2

Merged

Conversation

brian-smith-tcril
Copy link
Contributor

No description provided.

@bradenmacdonald
Copy link

Is this an alternative to openedx/frontend-component-footer#405 ?

@brian-smith-tcril brian-smith-tcril force-pushed the implement-footerslot branch 2 times, most recently from 1315ea0 to 99ac684 Compare May 14, 2024 15:46
@arbrandes
Copy link
Contributor

arbrandes commented May 14, 2024

@bradenmacdonald, I'd call it more of a precursor. We can still end up doing the things described there, but this is a simpler, more backward-compatible way (we hope, in particular for people that fork frontend-component-footer) to expose the footer as a slot - maybe in time for Redwood.

@brian-smith-tcril brian-smith-tcril force-pushed the implement-footerslot branch 6 times, most recently from aee12dc to e6e1a40 Compare May 15, 2024 18:07
@brian-smith-tcril brian-smith-tcril marked this pull request as ready for review May 15, 2024 19:08
Copy link
Contributor

@arbrandes arbrandes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@brian-smith-tcril brian-smith-tcril merged commit c6c9b09 into openedx:master May 15, 2024
4 checks passed
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

Successfully merging this pull request may close these issues.

3 participants