Skip to content

Fixing group access request todos appearance

Hitesh Raghuvanshi requested to merge 374608-fix-todos-group-access-request into master

What does this MR do and why?

This is a follow up MR of !103316 (merged). With the change in commit 42b431a2, the layout that was suggested changed. This MR does following:

  1. Fixes the UI layout to show the todos as expected.
  2. Fixes some issues in todos api while fetching member access requested todos.

Context: Whenever a user raises an access request for a group, todos will get created for recently active 10 owners of the group.

The todo will look like image

Screenshots or screen recordings

image

Also refer !103316 (merged)

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

  1. Create a public group named todo-request-check and make some users as its owner.
  2. Raise access request for the group from a non-member account.
  3. For the 10 recently active owners of group, todos should be created and they should be able to go to access requests page of the group.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #374608 (closed)

Edited by Hitesh Raghuvanshi

Merge request reports