On assignments allow the instructor to set a day/time for it to be make visible

[barbarer]

Right now I have to go into the instructor page and make my assignments for lecture visible just before or during lecture. It would be nice to have them automatically become visible at set day/time.

[jjjonesj]

I can take on this issue!

[bnmnetp]

I'm not sure what the best way to handle this is...

Right now we have a simple switch
If we add a date/time then we can just check that before showing the assignment. But what if you want to hide the assignment again, like after an exam? Just set the visible date to something far in the future?

It would be nice to not have both the switch and the date, and I really don't want to have both a show date and a hide date.

@barbarer @bhoffman0 What is the right flow here that will not confuse people?

[barbarer]

I agree that we need a show date and time and optionally a hide again date and time. Dr. Barbara Ericson Associate Professor, School of Information University of Michigan

…

On Fri, Jun 7, 2024 at 1:44 PM Bradley Miller ***@***.***> wrote: I'm not sure what the best way to handle this is... Right now we have a simple switch If we add a date/time then we can just check that before showing the assignment. But what if you want to hide the assignment again, like after an exam? Just set the visible date to something far in the future? It would be nice to not have both the switch and the date, and I really don't want to have both a show date and a hide date. @barbarer <https://github.com/barbarer> @bhoffman0 <https://github.com/bhoffman0> What is the right flow here that will not confuse people? — Reply to this email directly, view it on GitHub <#466 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AKOZ7MYPT433KCTAJWPCSBLZGHWRLAVCNFSM6AAAAABCMXJLZGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJVGI3DINRUGI> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[jjjonesj]

Hypothetically, wouldn't the hide date fall under the due date system? Or would we still have the same issue of having the calendar and switch in that case as well? If we want things to be automatic, the due date system would close the assignment once the specific time passes while the visibility just controls when it is seen, maybe?

[bnmnetp]

Every instructor has their own tolerance for how late to accept work. So you can't just hide the assignment on the due date or even X days after the due date.

In my case I said I would accept late work until I got around to grading it. So sometimes it was days and sometimes a week or more.

[bnmnetp]

My feeling is that we eliminate the switch, replacing it on the main panel with a date/time picker for when to make the assignment visible. It should default to the current time.

Then inside the "More Options" panel we add a "Hide After " date / time picker.

The logic in the chooseAssignment and doAssignment endpoints will need to be updated to use the dates if they are there, but fall back on the is_visible switch for old assignments created before we add this.

We don't have a background task to monitor the time to make something visible or invisible we just control what can be seen in the two endpoints we have for choosing and doing an assignment.

[jjjonesj]

I see now, that makes sense. I'll have to explore the files more for the "More Options" and to better understand eliminating the switch over the weekend. In the meantime, if we get rid of the switch for the new date system, how do we keep it partially intact for old assignments?

[bnmnetp]

The visibility of an assignment is represented in the model/database by the is_visible column. We will shift this to use visible_on or some other better name to indicate when an assignment should be visible.
There are probably two options:

1. remove the is_visible column from the model. This would force any existing systems to run an update that would set the visible_on time to utcnow if is_visible is True. If is_visible is False then set visible_on to 10 years from now.
2. keep the is_visible column around for the old interface, (still in use) but have the new interface just set the date.

Again, both the chooseAssignment and doAssignment endpoints will need to have their logic updated to use the new visible_on and/or hidden_on columns.

[jjjonesj]

So, where exactly do I start with that process? I think we should go with the second option if we're keeping previous assignments in mind. However, if it would be better for any future growth, it'd be best to go with the first option and just have everything updated. It's just a matter of what would benefit the platform overall and the easiest way of going about it at this point, I think.

[bnmnetp]

Well,

On the UI side:

• remove the is_visible switch
• add a date / time element for the visible_on value
• add a date / time element in the more options panel for hide_on
• Make sure all that saves properly for an assignment

On the server side:

• Update chooseAssignment and doAssignment

[jjjonesj]

Okay! I'm sure I can do both sides with guidance/instructions! I'll work the server side first since that's what my internship centers. I've been looking through the files and have seen that multiple contain chooseAssignment and doAssignment, which specific one do I need to update and how? Forgive me for asking so many questions, this is my first time working on the backend and being a student developer!

[bnmnetp]

What you should be looking at is in the student.py file under bases/rsptx/assignment_server_api

Anything under web2py is deprecated, and should not be changed or further developed.

[jjjonesj]

So, is it in these places that I should create a new variable "visible_on" with if statements to trigger the is_visible point?

[jjjonesj]

Hi, is it okay if we follow up on my previous question since you were absent yesterday? I'm still not quite sure how to update the logic. At first, I had changed the is_visible to visible_on but, I wasn't sure if that was correct and put it back. I was wondering if I needed to on create a new object visible_on using utcnow like the timestamp does.

[bnmnetp]

Its more than that, the visible_on value needs to be persistent, which means it needs to be stored in the database. So you are going to have to change the model to have a new column to keep track of the visible_on value for each assignment.

Then I think you should probably write a function that takes all the assignment data and returns true if the assignment should be visible and false if not.

It may help to follow the code down a couple of levels to help you understand what is going on.

It may also help you to go through the tutorial here: https://runestone-monorepo.readthedocs.io/en/latest/tutorial.html