Skip to content
Snippets Groups Projects

Resolve "Feedback page should use all feedbacks for summary"

What does this mr do?

See #660 (closed) for description and discussions

Screenshots

Empty Feedback

image

TAM View

image

Sorted by lastUpdatedAt (corresponds with the above image)

image

Manager View (summary includes all, but feedbacks only include courses managed)

image

(the review that states, "pretty mediocre" was an ADS request. This perspective is from an OOP manager)

Other things (stolen from issue desc)

image

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

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Ruben Backx
  • Ruben Backx
  • Henry Page added 8 commits

    added 8 commits

    Compare with previous version

  • Henry Page resolved all threads

    resolved all threads

  • Henry Page requested review from @rwbackx

    requested review from @rwbackx

  • Henry Page added 1 commit

    added 1 commit

    • 8dc5c1c5 - get rid of redundant comment

    Compare with previous version

  • Ruben Backx approved this merge request

    approved this merge request

  • Ruben Backx started a merge train

    started a merge train

  • merged

  • Ruben Backx mentioned in commit 79a6960c

    mentioned in commit 79a6960c

  • Please register or sign in to reply
    Loading