Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[#11737] Instructors reminding non-submitters: instructors are considered non-submitters even if there are no questions for instructors #11989

Closed
wants to merge 14 commits into from

Conversation

u6656793
Copy link

Fixes #11737

Outline of Solution

Logic problem I find
Instructors' submitted status is "No" when there is no question for them since the logic of getting non-submitters is that: getting set of students and instructors for a certain sessions, then getting a set of givers who submitted at least one question. The students and instructors who are not in the set of givers will be non-submitters.
How I solve it
I changed the way of finding submitters set. Since your logic of non-submitters is the one who is outside the givers set and those who have zero question in a session will be submitters, so those instructors are considered as givers. Then their submitted status will be "Yes".

@u6656793 u6656793 marked this pull request as draft October 29, 2022 14:20
@u6656793 u6656793 marked this pull request as ready for review October 29, 2022 15:38
@zhaojj2209 zhaojj2209 added the s.ToReview The PR is waiting for review(s) label Nov 2, 2022
@nusoss-bot
Copy link

Guys, This PR seems to be stalling (no activities for the past 7 days). 🐌 😢
Hope someone can get it to move forward again soon...

Copy link
Contributor

@zhaojj2209 zhaojj2209 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Apologies for the late response. This branch seems to revert a lot of changes on the master branch. Please ensure that the only changes on the branch are those related to the PR before we can review.

@nusoss-bot
Copy link

Guys, This PR seems to be stalling (no activities for the past 14 days). 🐌 😢
Hope someone can get it to move forward again soon...

@nusoss-bot
Copy link

Guys, This PR seems to be stalling (no activities for the past 16 days). 🐌 😢
Hope someone can get it to move forward again soon...

@nusoss-bot
Copy link

Guys, This PR seems to be stalling (no activities for the past 19 days). 🐌 😢
Hope someone can get it to move forward again soon...

@zhaojj2209
Copy link
Contributor

Closing due to inactivity. Feel free to reopen the PR if you would like to resume working on it.

@zhaojj2209 zhaojj2209 closed this Dec 6, 2022
@nusoss-bot
Copy link

Guys, This PR seems to be stalling (no activities for the past 23 days). 🐌 😢
Hope someone can get it to move forward again soon...

@nusoss-bot
Copy link

Guys, This PR seems to be stalling (no activities for the past 27 days). 🐌 😢
Hope someone can get it to move forward again soon...

@nusoss-bot
Copy link

Guys, This PR seems to be stalling (no activities for the past 30 days). 🐌 😢
Hope someone can get it to move forward again soon...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
s.ToReview The PR is waiting for review(s)
Projects
None yet
3 participants