Teammates: Instructor: edit feedback session: show course name and session status in edit form

[damithc]

Showing the session status (together with course ID and name) may be useful.

[arjunsuri]

Please guide me on this issue...

[huai99]

@damithc can i work on this issue ? :)

[whipermr5]

@huai99 Sure! You are always free to open a PR; we don't require asking :)

[huai99]

@damithc Hi, can you pinpoint to me in the UI where exactly you want to place the course ID and course name? That will be very helpful. Thanks.

[damithc]

@damithc Hi, can you pinpoint to me in the UI where exactly you want to place the course ID and course name? That will be very helpful. Thanks.

We can show the session status here:

The course ID is already there. you can show the course name just below the ID.

[HirdayGupta]

@damithc Sir, would this be affected by #7965?

[damithc]

@damithc Sir, would this be affected by #7965?

Yes, I guess so. Once #7965 is merged, this should follow the same.

[HirdayGupta]

@damithc Sir, Can I write a PR for this keeping the changes in #7965 in mind, and let it be on hold until it is merged?

[damithc]

@damithc Sir, Can I write a PR for this keeping the changes in #7965 in mind, and let it be on hold until it is merged?

Sure.

[huai99]

@HirdayGupta Hi, does that mean that you will be working on this issue ?

[HirdayGupta]

Hi @huai99, you are very welcome to work on this Issue if you wish to. You can choose to either wait until #7965 is merged or familiarise yourself with the contents of #7965 and write this Issue's PR accordingly. Since I already know how to do this, I am happy to assist you as well as review your PR to aid your transition into serious contributions for TEAMMATES.

[huai99]

@HirdayGupta Thanks for your help, i will first take some time to familiarize myself with #7965 and check from you if I have any questions. Thanks for being helpful. :)

[huai99]

@HirdayGupta In FeedbackSessionsForm, there is no attribute for the course name, so I will need to add a new attribute for the course name ? Or do you have any suggestions for a simpler approach?

[nidhi98gupta]

@damithc As per above comments since #7965 has been merged, can I start working on this? I mean is this issue still on hold?

[huai99]

@nidhi98gupta Hi, I have written a PR for this issue and I plan to submit latest by tomorrow, maybe you would like to look at other issues first at the moment? Sorry for the delay