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

enhance(macros/APIRef): show badges on all links #9757

Merged
merged 6 commits into from
May 2, 2024

Conversation

caugner
Copy link
Contributor

@caugner caugner commented Oct 2, 2023

Summary

Fixes #9430.

Problem

The APIRef macros was showing badges (e.g. deprecated) only on some links.

Solution

Add badges to the remaining links.


Screenshots

Before After
image image
image image

How did you test this change?

Ran yarn && yarn dev and opened http://localhost:3000/en-US/docs/Web/API/DOMException and http://localhost:3000/en-US/docs/Web/API/DOMError locally.

@github-actions github-actions bot added the macros tracking issues related to kumascript macros label Oct 2, 2023
@caugner caugner changed the title feat(macros/APIRef): show badges on all links enhance(macros/APIRef): show badges on all links Oct 2, 2023
@caugner caugner marked this pull request as draft October 2, 2023 14:40
@github-actions github-actions bot added the idle label Jan 24, 2024
@caugner caugner marked this pull request as ready for review February 29, 2024 22:45
@caugner caugner requested a review from a team as a code owner February 29, 2024 22:45
kumascript/macros/APIRef.ejs Outdated Show resolved Hide resolved
kumascript/macros/APIRef.ejs Outdated Show resolved Hide resolved
@caugner caugner force-pushed the APIRef-show-badges-on-all-links branch from 35748bd to 55971dd Compare February 29, 2024 22:47
@github-actions github-actions bot removed the idle label Feb 29, 2024
@github-actions github-actions bot added the merge conflicts 🚧 Please rebase onto or merge the latest main. label Apr 5, 2024
Copy link
Contributor

github-actions bot commented Apr 5, 2024

This pull request has merge conflicts that must be resolved before it can be merged.

@github-actions github-actions bot removed the merge conflicts 🚧 Please rebase onto or merge the latest main. label May 2, 2024
Copy link
Contributor Author

@caugner caugner left a comment

Choose a reason for hiding this comment

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

Self-approval to land this eventually.

@caugner caugner merged commit b7bbcaa into main May 2, 2024
14 checks passed
@caugner caugner deleted the APIRef-show-badges-on-all-links branch May 2, 2024 21:55
@hamishwillee
Copy link
Contributor

Thanks very much @caugner !!

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

Successfully merging this pull request may close these issues.

Deprecated: should be listed as deprecated in the sidebar
2 participants