-
Notifications
You must be signed in to change notification settings - Fork 18
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
Update index.rst #203
Update index.rst #203
Conversation
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.
FWIW I would note it higher up, but link to CITATION.md
instead. That way, we do not have to keep track of the same info in two different places.
Also, with Jenna on leave, who will approve this from ACS Team? @dvstark ?
Also preview is here: https://acstools--203.org.readthedocs.build/en/203/ |
I'm happy to fill in for Jenna. But when I look at the readthedocs preview, I don't see any specific link to the acstools citation, just a message that says "please cite acstools if you have used it in your work". Is this expected, or is there a way to put the actual link to the ads page right there in readthedocs? |
I see from the code Deep edited, the link is there, but it's not showing up in my preview |
Co-authored-by: P. L. Lim <2090236+pllim@users.noreply.github.com>
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.
Looks good to me!
testing citation preview