Skip to content

Fix broken links#620

Open
max-fw wants to merge 4 commits intomasterfrom
max-fw-issue522-broken-links
Open

Fix broken links#620
max-fw wants to merge 4 commits intomasterfrom
max-fw-issue522-broken-links

Conversation

@max-fw
Copy link
Collaborator

@max-fw max-fw commented Feb 5, 2026

  1. (On 3 pages) Added https:// to URLs where missing.

On steering-committee/index.md page:
2. Commented out html tags where individuals without a URL were defaulting to a broken link (a href="nan" would lead to broken link).
3. For Fotis' website link, I've entered a working URL instead of "nan".

Description

Fixes # (issue)

Type of Change

  • [ X] Breaking change (fix or feature that would cause existing functionality to not work as expected)

How Has This Been Tested?

Checklist for Content Editors and Non-Developers

  • The content is clear, accurate, and follows community guidelines.
  • All updated content has been previewed on the staging site.
  • All links, references, and formatting have been checked for correctness.
  • The change aligns with the overall style and communication goals.
  • No broken links in text/content

Checklist for Developers:

  • I have attempted to stay aligned to related code in this repository rather than reinventing the wheel.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no new warnings.

Additional Notes

1. Added https:// to URLs where missing. 
2. Commented out html <a> tags where individuals without a URL were defaulting to a broken link.
@github-actions github-actions bot added the content related Relevant to website content label Feb 5, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 5, 2026

✅ Spell Check Passed

No spelling issues found in this PR! 🎉

@github-actions
Copy link
Contributor

github-actions bot commented Feb 5, 2026

👍 All image files/references (if any) are in webp format, in line with our policy.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 5, 2026

✅ Spell Check Passed

No spelling issues found in this PR! 🎉

@max-fw max-fw changed the title Fix broken links on steering committee page Fix broken links Feb 5, 2026
Updated the link for the Institute for Replication (I4R) to the correct URL.
@github-actions
Copy link
Contributor

github-actions bot commented Feb 5, 2026

✅ Spell Check Passed

No spelling issues found in this PR! 🎉

@max-fw max-fw marked this pull request as ready for review February 5, 2026 11:52
@max-fw max-fw requested a review from a team as a code owner February 5, 2026 11:52
@LukasWallrich
Copy link
Contributor

Thanks @max-fw! The steering committee is generated here, so the broken links would be reintroduced on the next update. Could you implement the fixes here? https://github.com/forrtproject/forrtproject.github.io/blob/master/scripts/generate_sc_profiles.py

Great that you fixed the others (and found the issues).

Are these the only broken links? I would have expected more ...

@LukasWallrich
Copy link
Contributor

LukasWallrich commented Feb 5, 2026

Staging Deployment Status

This PR has been successfully deployed to staging as part of an aggregated deployment.

Deployed at: 2026-02-05 23:45:43 UTC
Staging URL: https://staging.forrt.org

The staging site shows the combined state of all compatible open PRs.

@max-fw
Copy link
Collaborator Author

max-fw commented Feb 5, 2026

We know which links are broken but it takes a while for me to locate where those links are appearing.

Question: This is a broken link:
https://forrt.org/self-assessment/
It seems to have been archived?

For other pages such as content/summaries/_index.md -- should I edit the .md file or is this generated from another .py file?

@richarddushime
Copy link
Contributor

We know which links are broken but it takes a while for me to locate where those links are appearing.

Question: This is a broken link: https://forrt.org/self-assessment/ It seems to have been archived?

For other pages such as content/summaries/_index.md -- should I edit the .md file or is this generated from another .py file?

Question: This is a broken link:
https://forrt.org/self-assessment/
It seems to have been archived?

the self assessment link and pages have been archived , incase you see them anywhere remove them

@github-actions
Copy link
Contributor

github-actions bot commented Feb 5, 2026

✅ Spell Check Passed

No spelling issues found in this PR! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

content related Relevant to website content

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants