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 a "finePrint" extension hook #1448

Merged
merged 3 commits into from
Feb 11, 2022
Merged

Add a "finePrint" extension hook #1448

merged 3 commits into from
Feb 11, 2022

Conversation

tsibley
Copy link
Member

@tsibley tsibley commented Jan 26, 2022

Description of proposed changes

Provides a bit of Markdown to render in the fine print at the bottom of
pages, including the splash page and dataset pages.

This will make it much easier to folks customizing Auspice to add their
own attributions, links to source code, etc.

Testing

Tested locally by building Auspice with and without configs that define finePrint.

It was documented later in the page but missing from the list at the
top.
Used by narratives too, not just the dataset footer.  About to be used
in the fine print component as well.
Provides a bit of Markdown to render in the fine print at the bottom of
pages, including the splash page and dataset pages.

This will make it much easier to folks customizing Auspice to add their
own attributions, links to source code, etc.
@tsibley tsibley requested a review from jameshadfield January 26, 2022 01:23
@nextstrain-bot nextstrain-bot temporarily deployed to auspice-trs-fine-print--hen1w6 January 26, 2022 01:23 Inactive
@tsibley tsibley requested a review from a team February 2, 2022 18:50
Copy link
Contributor

@joverlee521 joverlee521 left a comment

Choose a reason for hiding this comment

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

Easy win for a simple custom fine print!

@tsibley tsibley merged commit 8d13546 into master Feb 11, 2022
@tsibley tsibley deleted the trs/fine-print-hook branch February 11, 2022 22:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

3 participants