-
Notifications
You must be signed in to change notification settings - Fork 4
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
[DCJ-666] Update links to DUOS documentation to point to new location #2686
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.
This looks reasonable to me. I might suggest asking Victoria if she knows about those other links.
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 👍 , one minor suggestion:
The table below allows you to pre-authorize your Institution's users to request access to datasets, | ||
The table below allows you to | ||
<a href="https://support.terra.bio/hc/en-us/articles/28512587249051-How-to-Pre-Authorize-Researchers-to-Submit-Data-Access-Requests-in-DUOS" | ||
id="terra-support-pre-auth-link" target="_blank" rel="noreferrer"> pre-authorize </a> your Institution's users to request access to datasets, |
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.
id="terra-support-pre-auth-link" target="_blank" rel="noreferrer"> pre-authorize </a> your Institution's users to request access to datasets, | |
id="terra-support-pre-auth-link" target="_blank" rel="noreferrer">pre-authorize</a> your Institution's users to request access to datasets, |
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.
👍🏽 Thank you!
Addresses
https://broadworkbench.atlassian.net/browse/DCJ-666
Summary
I'm not exactly sure what the zendesk links in /ajax/Support.js (see search) are for and I couldn't find or access them on the UI so I didn't replace those. Every other instance of a zendesk link should be replaced thoughNever mind, they're API calls and not documentation links, so they won't be replacedVisual Aids
New "Help" and "pre-authorize" links:
Screen.Recording.2024-10-09.at.3.16.44.PM.mov
Have you read Terra's Contributing Guide lately? If not, do that first.