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

catalog.rst link updates #216

Merged
merged 11 commits into from
Jun 13, 2023
Merged

catalog.rst link updates #216

merged 11 commits into from
Jun 13, 2023

Conversation

smk0033
Copy link
Contributor

@smk0033 smk0033 commented Jun 1, 2023

Reference ticket: https://github.com/orgs/NASA-IMPACT/projects/29/views/6?pane=issue&itemId=29702274

Stac API link was previously updated for this ticket, but I also found other links that were broken, so this is a new PR to fix them.

First or second commit will be testing links as they're linked like "Searching STAC Documentation <search/searching_the_stac_catalog.ipynb>", so I'm adjusting layers to see what works.

@smk0033 smk0033 self-assigned this Jun 1, 2023
@smk0033 smk0033 requested a review from jjfrench June 1, 2023 16:23
@smk0033
Copy link
Contributor Author

smk0033 commented Jun 1, 2023

I think I fixed all of the links, let me know if I missed any! For the "Access" link, I just went ahead and linked the Access folder. "access.html" I think was what was originally there? Unless you think it'd be better for me to link the "accessing_external_data" file or one of those specifically @jjfrench

Copy link
Contributor

@jjfrench jjfrench left a comment

Choose a reason for hiding this comment

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

Great! Looks like the relative path works

@smk0033 smk0033 merged commit bd51aae into develop Jun 13, 2023
@smk0033 smk0033 deleted the catalog-link-updates branch June 13, 2023 20:46
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