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

broken URL fixes Nov 2024 #1353

Open
wants to merge 3 commits into
base: staging
Choose a base branch
from
Open

broken URL fixes Nov 2024 #1353

wants to merge 3 commits into from

Conversation

gufmar
Copy link
Collaborator

@gufmar gufmar commented Nov 13, 2024

  • clean up obsolete broken URLs
  • fix the raw .jsonld file link in the governance action example

Bonus: added the announcementBar switch in the docusaurus.config.js as just implemented on cardano.org

obsolete external domains
fix the raw .jsonld file link in the governance action example

Bonus: added the announcementBar switch like just implemented on cardano.org
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@gitmachtl or @disassembler
may I ask you to quickly check if this replacement of the URL is ok for the code examples?
Imo it still points to the same jsondld content, and it's just about to fix the link to it.

@gufmar gufmar requested a review from rphair November 14, 2024 10:28
Copy link
Collaborator

@rphair rphair left a comment

Choose a reason for hiding this comment

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

@gufmar I reviewed it to my own satisfaction yesterday but was waiting for a response to #1353 (comment) ... with no objection yet I believe this is good to merge as-is.

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.

3 participants