Skip to content

Json backlog testing#1878

Draft
cifarquhar wants to merge 11 commits into
mainfrom
json_backlog_testing
Draft

Json backlog testing#1878
cifarquhar wants to merge 11 commits into
mainfrom
json_backlog_testing

Conversation

@cifarquhar
Copy link
Copy Markdown
Contributor

DRAFT - Testing an alternative way of storing backlog issues

  • Adds org-cyf/data/backlog.json with title, category and url of current backlog issues
  • Updates issues-list-as-blocks.html to load the list of issues from this file instead of using the GitHub API
  • Updates backlog.html, backlog-issue.html and block.html to reflect the different data structure
  • Updates backlog/index.md for each ITP/SDC module to update labels
  • Removes the TV Show Sprint as these issues are now included across Data Flows
  • Updates success.md to display backlog titles correctly. Note: this means workshop titles do not display correctly. Leaving this for now while work is ongoing on Merge workshop content into currlicum #1822, I'll revisit once we have all the workshops in the same format.

Currently this works with a copy of the JSON generated by #1841. If a module wasn't included in that then it won't work here.

@netlify
Copy link
Copy Markdown

netlify Bot commented May 8, 2026

Deploy Preview for cyf-curriculum ready!

Name Link
🔨 Latest commit f75dbc1
🔍 Latest deploy log https://app.netlify.com/projects/cyf-curriculum/deploys/6a170c3d9681c80008d3593b
😎 Deploy Preview https://deploy-preview-1878--cyf-curriculum.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 88 (🔴 down 9 from production)
Accessibility: 100 (no change from production)
Best Practices: 92 (no change from production)
SEO: 100 (no change from production)
PWA: -
View the detailed breakdown and full score reports
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link
Copy Markdown

netlify Bot commented May 8, 2026

Deploy Preview for cyf-common ready!

Name Link
🔨 Latest commit f75dbc1
🔍 Latest deploy log https://app.netlify.com/projects/cyf-common/deploys/6a170c3dfd2b07000865d6b1
😎 Deploy Preview https://deploy-preview-1878--cyf-common.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Copy Markdown
Member

@illicitonion illicitonion left a comment

Choose a reason for hiding this comment

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

LGTM!

Presumably at some point we'll inline the issue text (so we don't have issue URLs any more), and do some auto-magic around generating e.g. the "how to submit" text rather than having it inline in each issue? (But before doing that, we either need to update the issue cloner to create issues from our source of truth rather than issues, or need to stop using coursework planners like we are)

*/}}
{{ range $repos }}
{{ $issueBlocks := partial "block/issues-list-as-blocks.html" (dict "backlog" . "backlog_filter" $backlog_filter "path" $currentPath) }}
{{ if eq 0 ($issueBlocks | len)}}
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is great for testing partial progress, but let's turn it into an error before merging

Comment on lines +11 to +12
{{ $courseRegex := `/(.*?)/` }}
{{ $courseName := (trim (index (findRE $courseRegex .path 1) 0) "/") }}
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think .Page.Section is the name of the course without needing to do path regex

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

Labels

None yet

Projects

Status: 📋 Backlog

Development

Successfully merging this pull request may close these issues.

2 participants