Skip to content

fix(curriculum): audit editable regions in tech conference schedule workshop#67779

Merged
majestic-owl448 merged 3 commits into
freeCodeCamp:mainfrom
jimjoice2804:fix-tech-conference-editable-regions
Jun 3, 2026
Merged

fix(curriculum): audit editable regions in tech conference schedule workshop#67779
majestic-owl448 merged 3 commits into
freeCodeCamp:mainfrom
jimjoice2804:fix-tech-conference-editable-regions

Conversation

@jimjoice2804
Copy link
Copy Markdown
Contributor

Checklist:

Refs #67721

Why I made this change

For new learners, it is better to keep the editable region as small and focused as possible. The previous editable region included <thead> and <tr>, but campers only need to edit the table header content.

Before 👇🏼

Screenshot 2026-06-03 at 10 25 10

After 👇🏼

Screenshot 2026-06-03 at 10 29 12

Copilot AI review requested due to automatic review settings June 3, 2026 06:31
@jimjoice2804 jimjoice2804 requested review from a team as code owners June 3, 2026 06:31
@github-actions github-actions Bot added scope: curriculum Lessons, Challenges, Projects and other Curricular Content in curriculum directory. deprioritized PR reviews are deprioritized; lacks productive input and ignores codebase best practices. labels Jun 3, 2026
@github-actions

This comment was marked as off-topic.

This comment was marked as off-topic.

@gikf gikf added status: waiting review To be applied to PR's that are ready for QA, especially when additional review is pending. Naomi's Sprints This label is for issues that Naomi is managing through her sprint initiatives on Discord. and removed deprioritized PR reviews are deprioritized; lacks productive input and ignores codebase best practices. labels Jun 3, 2026
@gikf
Copy link
Copy Markdown
Member

gikf commented Jun 3, 2026

Hey @jimjoice2804 at this point there's no need to update PR branch with new changes from main branch. Unless there appear merge conflicts, PR will be merged without any issues. All that it does right now is basically just starting again the CI checks.

@jimjoice2804
Copy link
Copy Markdown
Contributor Author

@gikf Thanks for the clarification. I’ll avoid updating the PR branch unless there are merge conflicts or a maintainer asks for changes.

@majestic-owl448 majestic-owl448 added status: waiting update Is awaiting update, after feedback or request for changes and removed status: waiting review To be applied to PR's that are ready for QA, especially when additional review is pending. labels Jun 3, 2026
…schedule/68c34329d430583a6cd5f7f1.md

Co-authored-by: majestic-owl448 <26656284+majestic-owl448@users.noreply.github.com>
Signed-off-by: Jimmy Joice <89591724+jimjoice2804@users.noreply.github.com>
@majestic-owl448 majestic-owl448 removed the status: waiting update Is awaiting update, after feedback or request for changes label Jun 3, 2026
Copy link
Copy Markdown
Contributor

@majestic-owl448 majestic-owl448 left a comment

Choose a reason for hiding this comment

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

Thank you for your contribution to the page! 👍
We are happy to accept these changes and look forward to future contributions. 🎉

@majestic-owl448 majestic-owl448 enabled auto-merge (squash) June 3, 2026 08:08
@majestic-owl448 majestic-owl448 merged commit 90290d9 into freeCodeCamp:main Jun 3, 2026
11 checks passed
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Jun 3, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Naomi's Sprints This label is for issues that Naomi is managing through her sprint initiatives on Discord. scope: curriculum Lessons, Challenges, Projects and other Curricular Content in curriculum directory.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants