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

DEV: Migrate custom header links to new objects setting type #53

Merged
merged 1 commit into from
Apr 23, 2024

Conversation

tgxworld
Copy link
Contributor

@tgxworld tgxworld commented Apr 4, 2024

Why this change?

Since discourse/discourse@a440e15, we have started to support objects typed theme setting so we are switching this theme component to use it instead as it provides a much better UX for configuring the settings required for the theme component.

@tgxworld tgxworld force-pushed the objects_schema branch 5 times, most recently from c8c973f to b617b0c Compare April 5, 2024 03:53
@tgxworld tgxworld force-pushed the objects_schema branch 2 times, most recently from 2a230e7 to 5c78ad1 Compare April 7, 2024 22:00
Why this change?

Since discourse/discourse@a440e15, we have started to support objects typed theme setting so we are switching this theme component to use it instead as it provides a much better UX for configuring the settings required for the theme component.
@tgxworld tgxworld merged commit 7374793 into main Apr 23, 2024
4 checks passed
@tgxworld tgxworld deleted the objects_schema branch April 23, 2024 03:50
OsamaSayegh added a commit that referenced this pull request Apr 24, 2024
OsamaSayegh added a commit that referenced this pull request Apr 24, 2024
OsamaSayegh added a commit that referenced this pull request Apr 24, 2024
OsamaSayegh added a commit that referenced this pull request Apr 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants