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

Materials of archived courses should be freezed/should not change #214

Open
messa opened this Issue Sep 30, 2017 · 4 comments

Comments

Projects
None yet
4 participants
@messa
Contributor

messa commented Sep 30, 2017

Right now courses, for example runs/2017/pyladies-praha-jaro/info.yml, just reference lessons by their path, for example beginners/testing.

What happens when the lesson moves to other directory or even disappears? It would break content of some previous course if someone doesn't dedicate their time to update all such references.

What happens when the lesson changes? I think the content of the old course should be preserved as is, as it was presented and teached.

Since all materials are currently in git on Github, accessible through git and Github APIs, this could be doable for example by creating a new branch after each course ends and material for the archived course could be rendered from the given branch instead of master.

@messa

This comment has been minimized.

Contributor

messa commented Sep 30, 2017

Looking from very high level, this is the same thing as #175 - the archived courses just would be rendered from branch in the upstream repository in same/similar way as contributed courses from PR branches.

@encukou encukou added the content label Oct 2, 2017

@encukou

This comment has been minimized.

Member

encukou commented Oct 2, 2017

Right, the plan is to do this after #175 is implemented. Thanks for saying it explicitly :)

@mikicz

This comment has been minimized.

Member

mikicz commented Apr 20, 2018

I removed the blocked label from this since #175 is practically finished.

@hroncok

This comment has been minimized.

Member

hroncok commented Apr 20, 2018

Note that MI-PYT runs are frozen and may serve as a reference.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment