-
Notifications
You must be signed in to change notification settings - Fork 129
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
export v2: footer description option #423
Conversation
Allows users to point to a Markdown file that contains description to display in the Auspice footer.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good @joverlee521, thanks! Please add a test build (e.g. fa98bb2 or similar) which uses a custom description/footer.
Can we update the documentation/migration guide as well? I suppose it's not officially part of v6 perhaps but this is the best description of all the export options we have, and even I find it useful as there are lots to remember...! |
Raise fatal error if the provided description file does not exist.
83de9aa
to
c8038db
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I accidentally typed in review of nextstrain/auspice#834 here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for addressing my comments, this looks good!
For
export v2
add a--description
option that takes a Markdown file and adds the text to the JSON inmeta.description
.This PR is dependent on nextstrain/auspice#834 to render the footer text in Auspice.