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

refactor(plugins/kuma): refactor custom block plugins specific to Kuma #8078

Conversation

bartsmykla
Copy link
Contributor

@bartsmykla bartsmykla commented Oct 23, 2024

Description

Removed separate tip.rb and warning.rb plugins, which were just copies of the same code with only difference in a block name. Now the code is the same and just registered for different tags.

It also correctly maps different class names between Kong and Kuma docs.

This is the followup on kumahq/kuma-website#2012

Testing instructions

Just go to any documentation page containing a warning or tip block to see if it renders the same as it was before

Preview link:

Checklist

@bartsmykla bartsmykla added the review:tech Request for technical review on the docs platform or processes. label Oct 23, 2024
@bartsmykla bartsmykla requested a review from a team as a code owner October 23, 2024 13:14
Copy link

netlify bot commented Oct 23, 2024

👷 Deploy request for kongdocs pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit ecc57ad

Remove separate `tip.rb` and `warning.rb` plugins, which were just copies of
the same code with only difference in a block name. Now the code is the same
and just registered for different tags.

It also correctly maps different class names between Kong and Kuma docs.

Signed-off-by: Bart Smykla <bartek@smykla.com>
@bartsmykla bartsmykla force-pushed the refactor/jekyll-kuma-custom-block-plugins branch from 6188e53 to ecc57ad Compare October 23, 2024 13:18
@bartsmykla
Copy link
Contributor Author

Closing in favor of: #8082

@bartsmykla bartsmykla closed this Oct 24, 2024
@bartsmykla bartsmykla deleted the refactor/jekyll-kuma-custom-block-plugins branch October 24, 2024 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
review:tech Request for technical review on the docs platform or processes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant