Closed johnperry99 closed 2 years ago
Hi @Davidy22 this is ready for your review whenever you're available
For now to merge this you can drop the commits related to #53 and create a seperate PR to resolve and finish closing screen.
Just checking in again, if you need this merged for your class you can drop the extraneous commits and I can merge the reformed PR, but I won't merge this with the end screen as-is because it introduces very undesired behavior if remaining workload lasts longer than the fixed length end screen
I'm going to close this so that a new PR with just the filters can be created. If you manage to produce a working version of the closing screen, I can look at that in a separate pr.
We've added the following filters for issue #69:
We also addressed #53 by creating a pretty screen for when saving at the end.