Skip to content

Fix #839: resolve remaining broken links needing manual action#840

Open
LukasWallrich wants to merge 2 commits into
mainfrom
fix/issue-839-broken-links-manual
Open

Fix #839: resolve remaining broken links needing manual action#840
LukasWallrich wants to merge 2 commits into
mainfrom
fix/issue-839-broken-links-manual

Conversation

@LukasWallrich

Copy link
Copy Markdown
Contributor

Resolves the manual-action items tracked in #839 (follow-up to #838 / #751).

Changes

Exclude column support (content/resources/resource.py)

  • Adds drop_excluded() so the generator honours the new first-column Exclude flag in the FORRT Database 2.0 sheet (gid 1924034107). Truthy cells (TRUE/1/yes/y/x) drop the row; blank cells are kept. Verified against the live sheet: 1491 → 1489 rows, the two flagged rows removed, Exclude column stripped from output.

Retired resources (excluded upstream, so removed here too)

  • hackathon-encouraging-open-science-pract.md — deleted Google Doc
  • preregistration-for-quantitative-researc.md — deleted Google Sheet

Link/metadata fixes

  • open-research-examples-of-good-practice.md → repointed to the OSF preprint https://doi.org/10.31219/osf.io/3r8hb (Farran, Silverstein, Ameen, Misheva & Gilmore), with updated author list and doi field.
  • unlikely-results.md → deleted Economist YouTube video replaced with the Dailymotion re-upload.
  • reversals.md → Patson et al. 2016 dead EBSCO session link replaced with its DOI https://doi.org/10.1037/xlm0000224 (verified via Crossref).

Removed page

  • content/replication-hub/fred_data_curators_role/ — the data-curators call, per maintainer decision. No remaining references in the repo.

Note on regeneration

The two curated-resource link edits (open-research-examples, unlikely-results) are also driven by the source sheet. Update the matching URL/DOI cells in the sheet so the fixes survive the daily regeneration; otherwise they'll revert on the next data-processing run.

🤖 Generated with Claude Code

- resource.py: honour the new "Exclude" column in the FORRT Database 2.0
  sheet, dropping flagged rows so retired resources aren't regenerated
- Remove the two curated resources now excluded upstream (deleted Google
  Docs/Sheet: hackathon, preregistration template)
- open-research-examples-of-good-practice: repoint to the OSF preprint
  (10.31219/osf.io/3r8hb) with updated author list
- unlikely-results: replace the deleted Economist YouTube video with the
  Dailymotion re-upload
- reversals: replace Patson et al. 2016 dead EBSCO link with its DOI
  (10.1037/xlm0000224)
- Remove the FRED data-curators role page

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@LukasWallrich
LukasWallrich requested a review from a team as a code owner July 22, 2026 15:53
@github-actions

Copy link
Copy Markdown
Contributor

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

@github-actions

Copy link
Copy Markdown
Contributor

📝 Spell Check Results

Found 1 potential spelling issue(s) when checking 4 changed file(s):

📄 content/reversals/reversals.md

Line Issue
2243 acn ==> can, acne

ℹ️ How to address these issues:

  1. Fix the typo: If it's a genuine typo, please correct it.
  2. Add to whitelist: If it's a valid word (e.g., a name, technical term), add it to .codespell-ignore.txt
  3. False positive: If this is a false positive, please report it in the PR comments.

🤖 This check was performed by codespell

@LukasWallrich

LukasWallrich commented Jul 22, 2026

Copy link
Copy Markdown
Contributor Author

Staging Deployment Status

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

Deployed at: 2026-07-22 18:42:45 UTC
Staging URL: https://staging.forrt.org

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

Matches the current OSF preprint title (10.31219/osf.io/3r8hb) and the
updated source-sheet cell.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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.

1 participant