search

RunestoneInteractive / rs

A New Monorepo structure for Runestone
Other
40 stars 66 forks source link

Workingon466 #503

Closed jjjonesj closed 2 months ago

jjjonesj commented 2 months ago

I made changes to the instructor.py by adding a hidden datetime object and switching the visible boolean to datetime under the new_Assignment function.

bnmnetp commented 2 months ago

We have talked about this before... You cannot remove the visible boolean or it will cause thousands of assignments to crash. In a live system backward compatibility is very important.

You and @kwizeras need to work together and give me one PR to look at. I can't try to look at two PRs trying to solve the same problem and figure out what is going on. There are lots of well documented workflows that you can find that will help you do this. I even suggested as much on @kwizeras 's PR.

Tagging @pearcej

PS - Sorry for the noise @creativecoder I got my discord and github handles mixed up.