Skip to content

Update link deployed after reach level 500 for TV-Show#1845

Open
Grajales-K wants to merge 3 commits into
CodeYourFuture:mainfrom
Grajales-K:UpdateNetlifyLevel500
Open

Update link deployed after reach level 500 for TV-Show#1845
Grajales-K wants to merge 3 commits into
CodeYourFuture:mainfrom
Grajales-K:UpdateNetlifyLevel500

Conversation

@Grajales-K
Copy link
Copy Markdown

Subject: Update Level 500 submission criteria to save Netlify credits

Description: Updated the instructions for the TV Show project. Students are now advised to deploy to Netlify only after reaching Level 500 to avoid exhausting build credits during early development.

@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 26, 2026

Deploy Preview for cyf-curriculum ready!

Name Link
🔨 Latest commit 543c224
🔍 Latest deploy log https://app.netlify.com/projects/cyf-curriculum/deploys/6a026b543a56bd000840a1bf
😎 Deploy Preview https://deploy-preview-1845--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.

@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 26, 2026

👷 Deploy request for cyf-common pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 543c224

Copy link
Copy Markdown
Contributor

@LonMcGregor LonMcGregor left a comment

Choose a reason for hiding this comment

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

This change sounds good. Would it be worth mentioning this in the readme for level 0 as well?

Clarified naming template for module issues.

Added visual separators to evidence list for better readability.

Standardized links for TV Show Project and Netlify deployment.
@Grajales-K
Copy link
Copy Markdown
Author

Grajales-K commented May 11, 2026

This change sounds good. Would it be worth mentioning this in the readme for level 0 as well?

Hi @LonMcGregor, I have made some changes. Regarding Netlify, it is now mentioned in both Level 0 and Level 500 to work on GitHub pages at first steps. Once they reach 500 level, they will use netlify, I also raised another PR that @illicitonion is checking.
If you want to have a look CodeYourFuture/Project-TV-Show#29

Regarding this issue, at this stage, students should have already read the instructions on how to deploy first on GitHub Pages. Once they reach Level 500 and all requirements are successfully completed, they perform the Netlify deployment as a final step.

So, in this PR, I have improved the formatting for better clarity and removed redundant info about reaching Netlify on Level 500.

I hope this helps. Thank you for your time reading and checking my changes.

@Grajales-K Grajales-K requested a review from LonMcGregor May 12, 2026 00:00
Copy link
Copy Markdown
Contributor

@LonMcGregor LonMcGregor left a comment

Choose a reason for hiding this comment

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

Thanks for making these changes

@github-project-automation github-project-automation Bot moved this from 📋 Backlog to 👀 In review in Curriculum Planning May 12, 2026
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