Skip to content

feat: add maximum submissions limit for forms#3199

Open
LAfricain wants to merge 14 commits intonextcloud:mainfrom
LAfricain:maxsubmissions
Open

feat: add maximum submissions limit for forms#3199
LAfricain wants to merge 14 commits intonextcloud:mainfrom
LAfricain:maxsubmissions

Conversation

@LAfricain
Copy link

Add the ability to limit the number of responses a form can receive. When the limit is reached, the form is automatically closed and displays a dedicated message instead of accepting new submissions.

  • Add max_submissions column to forms_v2_forms table (migration)
  • Add maxSubmissions property to Form entity
  • Check submission limit in FormsService::canSubmit()
  • Add limit enforcement in ApiController::newSubmission()
  • Add isMaxSubmissionsReached flag in form API response
  • Add limit settings UI in SettingsSidebarTab
  • Display dedicated 'Form is full' message in Submit view
  • Add French translations for new strings

Closes #596

Add the ability to limit the number of responses a form can receive.
When the limit is reached, the form is automatically closed and displays
a dedicated message instead of accepting new submissions.

- Add max_submissions column to forms_v2_forms table (migration)
- Add maxSubmissions property to Form entity
- Check submission limit in FormsService::canSubmit()
- Add limit enforcement in ApiController::newSubmission()
- Add isMaxSubmissionsReached flag in form API response
- Add limit settings UI in SettingsSidebarTab
- Display dedicated 'Form is full' message in Submit view
- Add French translations for new strings

Closes nextcloud#596
Copy link
Collaborator

@Chartman123 Chartman123 left a comment

Choose a reason for hiding this comment

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

@LAfricain thanks for your PR. I just had a quick look at your code and didn't test it yet in my instance. Already a few comments. :)

LAfricain and others added 8 commits March 3, 2026 19:03
Co-authored-by: Christian Hartmann <chris-hartmann@gmx.de>
Signed-off-by: L'Africain <fr.cyrille@tiberiade.be>
Co-authored-by: Christian Hartmann <chris-hartmann@gmx.de>
Signed-off-by: L'Africain <fr.cyrille@tiberiade.be>
Co-authored-by: Christian Hartmann <chris-hartmann@gmx.de>
Signed-off-by: L'Africain <fr.cyrille@tiberiade.be>
@Chartman123 Chartman123 added this to the 5.3 milestone Mar 3, 2026
Copy link
Collaborator

@Chartman123 Chartman123 left a comment

Choose a reason for hiding this comment

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

Some more comments :) Please have a look at the failing workflows, too.

lafricain79 and others added 5 commits March 4, 2026 08:48
Co-authored-by: Christian Hartmann <chris-hartmann@gmx.de>
Signed-off-by: L'Africain <fr.cyrille@tiberiade.be>
Co-authored-by: Christian Hartmann <chris-hartmann@gmx.de>
Signed-off-by: L'Africain <fr.cyrille@tiberiade.be>
Co-authored-by: Christian Hartmann <chris-hartmann@gmx.de>
Signed-off-by: L'Africain <fr.cyrille@tiberiade.be>
Co-authored-by: Christian Hartmann <chris-hartmann@gmx.de>
Signed-off-by: L'Africain <fr.cyrille@tiberiade.be>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Limit Number of Answers

2 participants