Resolve "Feedback page should use all feedbacks for summary"
All threads resolved!
What does this mr do?
See #660 (closed) for description and discussions
Screenshots
Empty Feedback
TAM View
lastUpdatedAt
(corresponds with the above image)
Sorted by
Manager View (summary includes all, but feedbacks only include courses managed)
(the review that states, "pretty mediocre" was an ADS request. This perspective is from an OOP manager)
Other things (stolen from issue desc)
Does this MR meet the acceptance criteria?
-
I have added a changelog entry to reflect the significant changes I made. -
Tests were created to test the feature. -
I have updated the documentation accordingly. -
I adhere to the style guide.
Edited by Henry Page
Merge request reports
Activity
added difficultylow priohigh statusdoing labels
Policy violation(s) detectedSecurity and compliance scanners enforced by your organization have completed and identified that approvals are required due to one or more policy violations. Review the policy's rules in the MR widget and assign reviewers to proceed. Learn more about Security and Compliance policies.
assigned to @hpage
- Resolved by Henry Page
- Resolved by Henry Page
- Resolved by Henry Page
added 8 commits
-
25ae85cb...7cd39805 - 5 commits from branch
development
- 428daabb - update feedback graphs
- 57387af8 - resolve mr conflicts
- c23af69a - add ruben feedback
Toggle commit list-
25ae85cb...7cd39805 - 5 commits from branch
requested review from @rwbackx
mentioned in commit 79a6960c
Please register or sign in to reply