DukeLearningInnovation / kits

https://dukelearninginnovation.github.io/kits
GNU General Public License v3.0
37 stars 5 forks source link

Combined Kits have no context #715

Closed joliet closed 2 years ago

joliet commented 2 years ago

I am instructor trying to find my kit from Spring 2020 but it is difficult to find because there is no organizational structure for combined kits. Since all of my courses are multi-section, all of my kits are combined kits and hard to navigate.

How might we (the Kits project team) better organize combined kits?

Questions

Assumptions

Hypotheses

joliet commented 2 years ago

How we might solve this:

noahhh commented 2 years ago

Hi @joliet , I've put this ticket on to test. I was working with very limited data locally so let me know how it looks to you.

joliet commented 2 years ago

This looks good @noahhh . I masqueraded as a couple of faculty I typically use for testing combined kits and those looked good. I added a few more combined kits and those organized correctly under the right semester. Is there anything else I can do to test this?

noahhh commented 2 years ago

@joliet I don't think so. Since we already weren't allowing non-sis courses to be combined, I don't think there's more to test? I am only checking for term and placing them in the term if there is a child in the term, not dealing with categories.

joliet commented 2 years ago

That makes sense @noahhh . I think this is working as expected. Not sure what Liz has that might need to go to prod as well. Let's talk about moving work ready for prod at the start of our meeting next Tuesday.

joliet commented 2 years ago

Closing so we know this one is ready for prod.

noahhh commented 2 years ago

@joliet this has been moved to production.