Skip to content

Fix broken/out-of-date docs links #2371

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
May 23, 2025
Merged

Fix broken/out-of-date docs links #2371

merged 1 commit into from
May 23, 2025

Conversation

NEStock
Copy link
Contributor

@NEStock NEStock commented May 9, 2025

  • Fixed a few links that were referencing the old "numbered" pages
  • Also replaced a few hardcoded references to https://docs.dandiarchive.org with the variable dandiDocumentationUrl

@NEStock NEStock mentioned this pull request May 9, 2025
Merged
@yarikoptic yarikoptic added UX Affects usability of the system patch Increment the patch version when merged vendorization labels May 23, 2025
yarikoptic
yarikoptic approved these changes May 23, 2025
View reviewed changes
Copy link
Member

@yarikoptic yarikoptic left a comment

Choose a reason for hiding this comment

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

Thank you! Looks good to me and on the preview. Permitted CI runs (since then invited you to collaborators, I assume should run automagically in the future)

@yarikoptic
Copy link
Member

frontend CI seems unhappy since does not have yet a fix @NEStock gave in another PR... let's proceed.

@yarikoptic yarikoptic merged commit 552912a into dandi:master May 23, 2025
2 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yarikoptic yarikoptic yarikoptic approved these changes

Assignees
No one assigned
Labels
patch Increment the patch version when merged UX Affects usability of the system vendorization
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@NEStock @yarikoptic