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

Fix db-less rendering in plugin hub #4190

Merged
merged 1 commit into from
Aug 5, 2022
Merged

Conversation

mheap
Copy link
Member

@mheap mheap commented Aug 5, 2022

Summary

Resolves #4161
Resolves #4094

Plugin hub was reporting that a plugin is db-less incompatible in the body, but that it's compatible in the sidebar.

Reason

The word yes is a boolean true in YAML, and the plugin specified yes rather than "yes" in the frontmatter. This PR adds a check for dbless_compatible == true in addition to the existing dbless_compatible == "yes"

Testing

/hub/smartparkingtechnology/google-logging/

Old:
image

New:
image

@mheap mheap added the review:general Review for general accuracy and presentation. Does the doc work? Does it output correctly? label Aug 5, 2022
@mheap mheap requested a review from a team as a code owner August 5, 2022 10:17
@netlify
Copy link

netlify bot commented Aug 5, 2022

Deploy Preview for kongdocs ready!

Name Link
🔨 Latest commit 844910a
🔍 Latest deploy log https://app.netlify.com/sites/kongdocs/deploys/62ecee3d9772e90009e0ae5d
😎 Deploy Preview https://deploy-preview-4190--kongdocs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@cloudjumpercat cloudjumpercat self-assigned this Aug 5, 2022
@cloudjumpercat cloudjumpercat merged commit c440626 into main Aug 5, 2022
@cloudjumpercat cloudjumpercat deleted the google-logging-dbless branch August 5, 2022 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
review:general Review for general accuracy and presentation. Does the doc work? Does it output correctly?
Projects
None yet
2 participants