search

open-sauced / app

🍕 Insights into your entire open source ecosystem.
https://pizza.new
Apache License 2.0
414 stars 222 forks source link

Feature: Enable add member button and present upgrade modal if over the limit #3534

Open nickytonline opened 3 months ago

nickytonline commented 3 months ago 
          @brandonroberts, just getting to review this. I think the "add members" should present the modal and not be disabled. That thought might present some extra UX decisions, so I am approving this, but would love to at minimum follow up on this in an issue.

Screenshot 2024-06-06 at 2 28 48 PM

Originally posted by @bdougie in https://github.com/open-sauced/app/pull/3511#pullrequestreview-2103249976

github-actions[bot] commented 3 months ago

Thanks for the issue, our team will look into it as soon as possible! If you would like to work on this issue, please wait for us to decide if it's ready. The issue will be ready to work on once we remove the "needs triage" label.

To claim an issue that does not have the "needs triage" label, please leave a comment that says ".take". If you have any questions, please reach out to us on Discord or follow up on the issue itself.

For full info on how to contribute, please check out our contributors guide.

brandonroberts commented 3 months ago

Thanks. Currently, the Add Member button is disabled only because the username text field is empty.