Skip to content

Fixed an issue where min number of users in purchase flow was incorrect

Vamsi Vempati requested to merge 352231-purchase-workflow-users-count-fix into master

What does this MR do and why?

Fixes an issue where SaaS purchase workflow set the minimum user count to be the group member count without taking into account the users in the hierarchy

Screenshots or screen recordings

Setup Screenshot
Group with say 5 members Screen_Shot_2022-02-17_at_3.08.38_PM
Project in the group with say 10 members Screen_Shot_2022-02-17_at_3.08.46_PM
Before After
Screen_Shot_2022-02-17_at_3.14.28_PM Screen_Shot_2022-02-17_at_3.09.58_PM

How to set up and validate locally

  1. Log in to GitLab
  2. Go to a group thats on free plan, if no such group exists, create a new one
  3. Set up the group with x users and with a project with > x users
  4. Go to the group's Billing page
  5. Upgrade the group -> we should be taken to subscription purchase flow

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 #352231

Edited by Vamsi Vempati

Merge request reports