Skip to content

feat: create modal limit alerting #4429

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 12 commits into from
Sep 28, 2025
Merged

feat: create modal limit alerting #4429

merged 12 commits into from
Sep 28, 2025

Conversation

IMB11
Copy link
Member

@IMB11 IMB11 commented Sep 27, 2025 •
edited
Loading

  • Implements a new alert that shows when the user has used up 75% of their limits for creating projects, collections or orgs.
  • Replaces the DropdownSelect with Chips in the ProjectCreateModal which is much cleaner.
  • Moved all the create modals into the ui/create folder.
  • i18nify'd the whole shebang

@IMB11 IMB11 added enhancement New feature or request frontend labels Sep 27, 2025
@IMB11 IMB11 requested a review from Prospector September 27, 2025 16:41
@IMB11 IMB11 marked this pull request as ready for review September 27, 2025 16:41
@IMB11 IMB11 enabled auto-merge September 27, 2025 18:34
@IMB11 IMB11 disabled auto-merge September 28, 2025 00:05
@IMB11 IMB11 enabled auto-merge September 28, 2025 16:15
Prospector
Prospector approved these changes Sep 28, 2025
View reviewed changes
apps/frontend/src/components/ui/create/ProjectCreateModal.vue
<span>{{ formatMessage(messages.visibilityDescription) }}</span>
</label>
<DropdownSelect
<Chips
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i'd kinda like these to be radio buttons in the future since I think that style looks nicer, but for now this is fine

@IMB11 IMB11 added this pull request to the merge queue Sep 28, 2025
Merged via the queue into main with commit d418eae Sep 28, 2025
2 checks passed
@Prospector Prospector deleted the cal/dev-288 branch September 29, 2025 18:07
fetchfern pushed a commit that referenced this pull request Oct 1, 2025
@IMB11 @fetchfern
feat: create modal limit alerting (#4429)
723ad3e 
* draft: layout for alert

* feat: simplify

* feat: remove dummy data

* fix: lint and widths

* feat: use chips rather than dropdown select

* feat: remove gap from admonition header v body

* Revert "feat: remove gap from admonition header v body"

This reverts commit 46cce52.

* fix: niche fixes

* feat: update for new backend structure

* fix: i18n
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Prospector Prospector Prospector approved these changes

Assignees
No one assigned
Labels
enhancement New feature or request frontend
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@IMB11 @Prospector