Skip to content
Snippets Groups Projects

Add empty state to pending members page

Merged Ammar Alakkad requested to merge 438719-add-empty-state-to-pending-members into master
All threads resolved!

What does this MR do and why?

The MR adds an empty state to the pending members page when there are no pending members left to approve.

note: no changelog since the change is behind a feature flag.

Related to #438719 (closed)

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

Before After
Screenshot_2024-02-22_at_11.31.37 Screenshot_2024-02-22_at_11.31.00

How to set up and validate locally

  1. Enable saas_user_caps feature flag
  2. Run GDK in SaaS mode
  3. As admin/owner, navigate to a group's members page (Group page > Manage > Members)
  4. Using the alert in the members page, navigate to the pending members page
  5. If the page is empty, you should see the empty state introduced by this MR (otherwise, approve all pending members to see the empty state)
Edited by Ammar Alakkad

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
  • Ammar Alakkad changed milestone to %16.10

    changed milestone to %16.10

  • assigned to @aalakkad

  • Contributor
    1 Message
    :book: CHANGELOG missing:

    If this merge request needs a changelog entry, add the Changelog trailer to the commit message you want to add to the changelog.

    If this merge request doesn't need a CHANGELOG entry, feel free to ignore this message.

    Reviewer roulette

    Category Reviewer Maintainer
    frontend @mcavoj profile link current availability (UTC+1, 2 hours behind author) @himkp profile link current availability (UTC+1, 2 hours behind author)

    Please check reviewer's status!

    • available Reviewer is available!
    • unavailable Reviewer is unavailable!

    Please refer to documentation page for guidance on how you can benefit from the Reviewer Roulette, or use the GitLab Review Workload Dashboard to find other available reviewers.

    If needed, you can retry the :repeat: danger-review job that generated this comment.

    Generated by :no_entry_sign: Danger

    • Author Developer
      Resolved by Rahul Chanila

      Hey @alex_martin, I got two options for the image size that I need to get PM input which one to pick

      default height 220px height todos page for reference
      Screenshot_2024-02-21_at_09.50.01 Screenshot_2024-02-21_at_09.49.48 Screenshot_2024-02-21_at_09.54.57

      In the todos page when there is not todos left, we should the same image and heading but we also add 2 lines. The idea behind increasing the height is to compensate for extra the whitespace in the page.

      Also, do we need to make any adjustments for the copy or it's good as it's now for a first iteration?

  • Contributor

    Bundle size analysis [beta]

    This compares changes in bundle size for entry points between the commits 03889419 and a8aaafda

    :sparkles: Special assets

    Entrypoint / Name Size before Size after Diff Diff in percent
    average 4.21 MB 4.21 MB - 0.0 %
    mainChunk 3.21 MB 3.21 MB - 0.0 %

    Please look at the full report for more details


    Read more about how this report works.

    Generated by :no_entry_sign: Danger

  • Ammar Alakkad added 1 commit

    added 1 commit

    • a8252d5b - Add empty state to pending members page

    Compare with previous version

  • Ammar Alakkad changed the description

    changed the description

  • Contributor

    Quarantined Fulfillment QA E2E Tests

    The following Fulfillment QA E2E tests are in quarantine. If your changes would be covered by these tests, please be sure to perform manual testing.

    Description Location
    Fulfillment Utilization when admin sets user cap limit for group shows members over limit as pending for approvals ./qa/specs/features/ee/browser_ui/11_fulfillment/utilization/saas_user_caps_spec.rb[1:1:1:1]
    Fulfillment with existing compute minutes packs behaves like Purchase storage adds additional storage to group namespace ./qa/specs/features/ee/browser_ui/11_fulfillment/purchase/purchase_storage_spec.rb[1:3:1:1]
    Fulfillment with an active subscription behaves like Purchase storage adds additional storage to group namespace ./qa/specs/features/ee/browser_ui/11_fulfillment/purchase/purchase_storage_spec.rb[1:2:1:1]
    Fulfillment without active subscription behaves like Purchase storage adds additional storage to group namespace ./qa/specs/features/ee/browser_ui/11_fulfillment/purchase/purchase_storage_spec.rb[1:1:1:1]
  • Ammar Alakkad added 1 commit

    added 1 commit

    • b75e179f - Add empty state to pending members page

    Compare with previous version

  • Ammar Alakkad changed the description

    changed the description

  • requested review from @minahilnichols

  • Fiona Neill approved this merge request

    approved this merge request

  • Contributor

    E2E Test Result Summary

    allure-report-publisher generated test report!

    e2e-test-on-gdk: :white_check_mark: test report for a8aaafda

    expand test summary
    +------------------------------------------------------------------+
    |                          suites summary                          |
    +-------------+--------+--------+---------+-------+-------+--------+
    |             | passed | failed | skipped | flaky | total | result |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Govern      | 3      | 0      | 0       | 0     | 3     | ✅     |
    | Data Stores | 2      | 0      | 0       | 0     | 2     | ✅     |
    | Monitor     | 4      | 0      | 0       | 0     | 4     | ✅     |
    | Create      | 8      | 0      | 3       | 0     | 11    | ✅     |
    | Plan        | 4      | 0      | 0       | 0     | 4     | ✅     |
    | Package     | 0      | 0      | 1       | 0     | 1     | ➖     |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Total       | 21     | 0      | 4       | 0     | 25    | ✅     |
    +-------------+--------+--------+---------+-------+-------+--------+

    e2e-review-qa: :white_check_mark: test report for b75e179f

    expand test summary
    +------------------------------------------------------------------+
    |                          suites summary                          |
    +-------------+--------+--------+---------+-------+-------+--------+
    |             | passed | failed | skipped | flaky | total | result |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Create      | 8      | 0      | 3       | 3     | 11    | ✅     |
    | Govern      | 3      | 0      | 0       | 0     | 3     | ✅     |
    | Data Stores | 2      | 0      | 0       | 0     | 2     | ✅     |
    | Plan        | 3      | 0      | 1       | 0     | 4     | ✅     |
    | Monitor     | 4      | 0      | 0       | 0     | 4     | ✅     |
    | Package     | 0      | 0      | 1       | 0     | 1     | ➖     |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Total       | 20     | 0      | 5       | 3     | 25    | ✅     |
    +-------------+--------+--------+---------+-------+-------+--------+

    e2e-package-and-test: :white_check_mark: test report for a8aaafda

    expand test summary
    +------------------------------------------------------------------+
    |                          suites summary                          |
    +-------------+--------+--------+---------+-------+-------+--------+
    |             | passed | failed | skipped | flaky | total | result |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Fulfillment | 8      | 0      | 75      | 4     | 83    | ✅     |
    | Create      | 153    | 0      | 19      | 4     | 172   | ✅     |
    | Monitor     | 8      | 0      | 0       | 0     | 8     | ✅     |
    | Plan        | 8      | 0      | 0       | 0     | 8     | ✅     |
    | Govern      | 6      | 0      | 0       | 0     | 6     | ✅     |
    | Data Stores | 4      | 0      | 0       | 0     | 4     | ✅     |
    | Package     | 0      | 0      | 2       | 0     | 2     | ➖     |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Total       | 187    | 0      | 96      | 8     | 283   | ✅     |
    +-------------+--------+--------+---------+-------+-------+--------+
  • Minahil Nichols approved this merge request

    approved this merge request

  • Minahil Nichols requested review from @rchanila and removed review request for @minahilnichols

    requested review from @rchanila and removed review request for @minahilnichols

  • Ammar Alakkad added 983 commits

    added 983 commits

    Compare with previous version

  • Ammar Alakkad requested review from @rchanila

    requested review from @rchanila

  • Ammar Alakkad added 1 commit

    added 1 commit

    • a8aaafda - Translate empty state title prop

    Compare with previous version

  • Rahul Chanila approved this merge request

    approved this merge request

  • Rahul Chanila resolved all threads

    resolved all threads

  • merged

  • Rahul Chanila mentioned in commit b6701612

    mentioned in commit b6701612

  • added workflowstaging label and removed workflowcanary label

  • Please register or sign in to reply
    Loading