CodeYourFuture / syllabus

Old Syllabus Website for CodeYourFuture
https://syllabus.codeyourfuture.io
154 stars 124 forks source link

Revert "PD-JS1" #586

Closed SallyMcGrath closed 1 year ago

SallyMcGrath commented 1 year ago

Reverts CodeYourFuture/syllabus#581

SallyMcGrath commented 1 year ago

@esma-g @kfklein15

Some quite large (fixable) issues with the PD changes. Please slow down as you are merging with another team.

Chiefly here:

-- you have named the new files learning-objective but the existing matched files are called learning-objectives (https://syllabus.codeyourfuture.io/html-css/week-1/learning-objectives)

-- I don't think the learning objectives given here meet the Teach Tech Together standards for learning objectives and this is a rule for our syllabus. We can't accept learning objectives that don't meet the minimum standard. Here's the reference:

https://teachtogether.tech/en/index.html#s:process-objectives

esma-g commented 1 year ago

Thanks Sally. I have created new PRs for the PD content for each week in JS1 module, fixing the issue and excluding the learning objectives as discussed here.

(@kfklein15 fyi)