Skip to content

SDC Types Rework - #2018

Open
LonMcGregor wants to merge 35 commits into
mainfrom
lm-sdc-tools-5-rework
Open

LonMcGregor wants to merge 35 commits into
mainfrom
lm-sdc-tools-5-rework

Conversation

@LonMcGregor

@LonMcGregor LonMcGregor commented Sep 23, 2026 •

Copy link
Copy Markdown
Contributor

Reworks the content in SDC Types.
Associated PR for the tasks: CodeYourFuture/Module-Tools#686
Builds on the work started in #1967

Particularly looking for review on:

  • Whether you think the reworked tasks are appropriately presented
  • If the language used is consistent, and not too dense

Additional tasks todo:

  • Update the backlog tickets issue descriptions
  • Update the portal tasks issue descriptions
  • Update the solutions branch
  • Update review instructions for reviewers

@netlify

netlify Bot commented Sep 23, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for cyf-common ready!

Name Link
🔨 Latest commit 2a363b9
🔍 Latest deploy log https://app.netlify.com/projects/cyf-common/deploys/6ac3c3fc59e1180008c6b2af
😎 Deploy Preview https://deploy-preview-2018--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.

@netlify

netlify Bot commented Sep 23, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for cyf-curriculum ready!

Name Link
🔨 Latest commit 2a363b9
🔍 Latest deploy log https://app.netlify.com/projects/cyf-curriculum/deploys/6ac3c3fcca188e00088b725e
😎 Deploy Preview https://deploy-preview-2018--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: 85 (no change 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.

@LonMcGregor
LonMcGregor marked this pull request as ready for review September 30, 2026 15:03

@illicitonion illicitonion left a comment

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 looks great, much clearer, thank you!

Comment thread common-content/en/module/decomposition/why-types/index.md Outdated
Comment thread common-content/en/module/decomposition/why-types/index.md Outdated
Comment thread common-content/en/module/decomposition/why-types/index.md Outdated
Comment thread common-content/en/module/decomposition/classes-and-objects/index.md Outdated
Comment thread common-content/en/module/decomposition/methods/index.md Outdated
Comment thread common-content/en/module/decomposition/enums/index.md Outdated
Comment thread common-content/en/module/decomposition/inheritance/index.md Outdated
Comment thread common-content/en/module/decomposition/inheritance/index.md Outdated
Comment thread common-content/en/module/decomposition/inheritance/index.md
Comment thread common-content/en/module/decomposition/inheritance/index.md Outdated
…ex.md

Co-authored-by: Daniel Wagner-Hall <dawagner@gmail.com>
l and others added 12 commits October 5, 2026 10:41
Co-authored-by: Daniel Wagner-Hall <dawagner@gmail.com>
Co-authored-by: Daniel Wagner-Hall <dawagner@gmail.com>
Co-authored-by: Daniel Wagner-Hall <dawagner@gmail.com>
Co-authored-by: Daniel Wagner-Hall <dawagner@gmail.com>
Co-authored-by: Daniel Wagner-Hall <dawagner@gmail.com>
Co-authored-by: Daniel Wagner-Hall <dawagner@gmail.com>
Co-authored-by: Daniel Wagner-Hall <dawagner@gmail.com>
Co-authored-by: Daniel Wagner-Hall <dawagner@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 👀 In review

Development

Successfully merging this pull request may close these issues.

2 participants