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

Add show_review_banner configuration option #122

Merged
merged 2 commits into from
Oct 11, 2019
Merged

Conversation

vixus0
Copy link
Contributor

@vixus0 vixus0 commented Oct 11, 2019

Previously, disabling the review banner that appears at the bottom of pages
required a weird JavaScript hack. I've added a new config option that will
allow users to disable the banner globally.

I've changed the `rebuild_site!` function to take an `overrides` argument which is
a hash of options that will be added to the tech-docs config. Now you
don't need to copy/paste the entire YAML config just to change a single
option!
Previously, disabling the review banner that appears at the bottom of
pages required a weird JavaScript hack. I've added a new config option
that will allow users to disable the banner globally.
Copy link

@bravegrape bravegrape left a comment

Choose a reason for hiding this comment

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

I've tested the new show_review_banner feature in combination with the show_expiry global config and it gives the expected results in all 4 cases 👍

@vixus0 vixus0 merged commit 0e54d52 into master Oct 11, 2019
@vixus0 vixus0 deleted the hide-review-banner branch October 11, 2019 14:39
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.

2 participants