-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
[#10508] Instructor courses page: some items don't show spinners #12521
[#10508] Instructor courses page: some items don't show spinners #12521
Conversation
Hi @rexong, thank you for your interest in contributing to TEAMMATES!
Please address the above before we proceed to review your PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @rexong, looks great (: Only one small nit: could you change isLoadingSoftDeleteCourses
to isLoadingSoftDeletedCourses
? I think this is more consistent with the other Loading
variables like isLoadingArchivedCourses
as well as variables such as softDeletedCourses
and softDeletedCourse
Can I check if there is any updates on the E2E Test? I cant seem to pass the E2E test but not sure why. |
I've just merged in a PR that fixes this issue - once you update your branch (and the branch for your other PR), it should be fine |
I have updated my branch and made the relevant changes. Ready for review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Hello, I think this PR still requires another review? Thank you! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Fixes #10508
Outline of Solution
Active Courses
while displaying the spinner.TEAMMATES.-.Online.Peer.Feedback_Evaluation.System.for.Student.Team.Projects.Mozilla.Firefox.2023-07-23.19-57-50.mp4