Skip to content

Improve dispatcher - #63326

Merged
CarlSchwan merged 3 commits into
masterfrom
carl/dispatcher
Aug 18, 2026
Merged

Improve dispatcher#63326
CarlSchwan merged 3 commits into
masterfrom
carl/dispatcher

Conversation

@CarlSchwan

Copy link
Copy Markdown
Member

Summary

Do various level of verification of the data sent to controller methods based on the phpdoc/psalm comments + support backed enums

TODO

  • ...

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

Support for various psalm scalar types: non-empty-string, positive-int,
...

Signed-off-by: Carl Schwan <carl@carlschwan.eu>
Assisted-by: ClaudeCode:claude-opus-5
Signed-off-by: Carl Schwan <carl@carlschwan.eu>
@CarlSchwan CarlSchwan added this to the Nextcloud 35 milestone Aug 17, 2026
@CarlSchwan CarlSchwan self-assigned this Aug 17, 2026
@CarlSchwan
CarlSchwan requested a review from a team as a code owner August 17, 2026 21:17
@CarlSchwan CarlSchwan added 3. to review Waiting for reviews technical debt 🧱 🤔🚀 labels Aug 17, 2026
@CarlSchwan
CarlSchwan requested review from come-nc, leftybournes, salmart-dev and sorbaugh and removed request for a team August 17, 2026 21:17
Better than a generic 500

Signed-off-by: Carl Schwan <carl@carlschwan.eu>
@nextcloud-bot nextcloud-bot mentioned this pull request Aug 18, 2026

@szaimen szaimen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🐘

@CarlSchwan
CarlSchwan merged commit 65f77f2 into master Aug 18, 2026
210 of 212 checks passed
@CarlSchwan
CarlSchwan deleted the carl/dispatcher branch August 18, 2026 11:41
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.

4 participants