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

Rework tracks grouping/presentation #1170

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Rework tracks grouping/presentation #1170

wants to merge 3 commits into from

Conversation

illicitonion
Copy link
Member

What does this change?

As per our last syllabus team meeting, we think these are the three families of tracks we have.

Explain a bit of context as to what each category is. This maybe makes more sense to do under the headings of each group of tracks? But then the text is going to be awfully skinny...

Also, self-study tracks may have specific prereqs - we should list them so people know what they may want to opt to study.

Common Theme?

  • Adds the ability to show prereqs on module cards
  • Needs some styling on the card display

Org Content?

Updates tracks metadata.

Copy link

netlify bot commented Nov 4, 2024

Deploy Preview for cyf-curriculum canceled.

Name Link
🔨 Latest commit d78b01d
🔍 Latest deploy log https://app.netlify.com/sites/cyf-curriculum/deploys/672a2303577c4e0008c13910

Copy link

netlify bot commented Nov 4, 2024

Deploy Preview for cyf-common canceled.

Name Link
🔨 Latest commit d78b01d
🔍 Latest deploy log https://app.netlify.com/sites/cyf-common/deploys/672a2303fdfa3d0008c4e077

Copy link

netlify bot commented Nov 4, 2024

Deploy Preview for cyf-launch ready!

Name Link
🔨 Latest commit d78b01d
🔍 Latest deploy log https://app.netlify.com/sites/cyf-launch/deploys/672a230330c4e8000828646e
😎 Deploy Preview https://deploy-preview-1170--cyf-launch.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

Copy link

netlify bot commented Nov 4, 2024

Deploy Preview for cyf-sdc ready!

Name Link
🔨 Latest commit d78b01d
🔍 Latest deploy log https://app.netlify.com/sites/cyf-sdc/deploys/672a230376fe9e0008050267
😎 Deploy Preview https://deploy-preview-1170--cyf-sdc.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

Copy link

netlify bot commented Nov 4, 2024

Deploy Preview for cyf-programming ready!

Name Link
🔨 Latest commit d78b01d
🔍 Latest deploy log https://app.netlify.com/sites/cyf-programming/deploys/672a2303b78176000897f832
😎 Deploy Preview https://deploy-preview-1170--cyf-programming.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: 99 (🟢 up 1 from production)
Accessibility: 100 (no change from production)
Best Practices: 92 (no change from production)
SEO: 90 (no change from production)
PWA: -
View the detailed breakdown and full score reports

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

Copy link

netlify bot commented Nov 4, 2024

Deploy Preview for cyf-tracks ready!

Name Link
🔨 Latest commit d78b01d
🔍 Latest deploy log https://app.netlify.com/sites/cyf-tracks/deploys/672a23033c0b440008202b01
😎 Deploy Preview https://deploy-preview-1170--cyf-tracks.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

Copy link

netlify bot commented Nov 4, 2024

Deploy Preview for cyf-piscine ready!

Name Link
🔨 Latest commit d78b01d
🔍 Latest deploy log https://app.netlify.com/sites/cyf-piscine/deploys/672a2303f0f1670008ef19ba
😎 Deploy Preview https://deploy-preview-1170--cyf-piscine.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: 85 (🔴 down 15 from production)
Accessibility: 100 (no change from production)
Best Practices: 100 (no change from production)
SEO: 100 (no change from production)
PWA: 60 (no change from production)
View the detailed breakdown and full score reports

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

@illicitonion illicitonion mentioned this pull request Nov 4, 2024
5 tasks
Copy link

netlify bot commented Nov 4, 2024

Deploy Preview for cyf-itd canceled.

Name Link
🔨 Latest commit d78b01d
🔍 Latest deploy log https://app.netlify.com/sites/cyf-itd/deploys/672a2303b7a0790008fb0ca1

@SallyMcGrath
Copy link
Member

Oh Daniel do you want to break my heart! Absolutely not! 😂 😭

I mean, I completely agree with the organisation and the descriptions, but we can't have this layout I will DIE. I think it might another case for Taxonomies?

And we can put some kind of info dump underneath if we need to.

As per our last syllabus team meeting, we think these are the three
families of tracks we have.

Explain a bit of context as to what each category is. This maybe makes
more sense to do under the headings of each group of tracks? But then
the text is going to be awfully skinny...

Also, self-study tracks may have specific prereqs - we should list them
so people know what they may want to opt to study.
Some things we should decide:
* Do we want to still support menus, or should we just make everything
  be a taxonomy?
* How do we want to handle compatibility here? I've renamed the map
  frontmatter to menus_to_map to make it more clear what it is, but this
  is a breaking change - I can add in a fallback for compatibility, or
  just have this be a break. If we're deleting the menus way and forcing
  taxonomies, we should almost certainly do that as a breaking change.
  If we're going to allow both, it's more of an open question.

And we need to style the new layout, because it's hideous.
@illicitonion illicitonion marked this pull request as draft November 9, 2024 13:37
@illicitonion
Copy link
Member Author

I've pulled out the "adding a taxonomy map" into #1187 and will pick up the "adding metadata to the terms" piece here when that is merged.

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