Skip to content

Add modal field validation using Marmalade Validator#6

Merged
Aerhhh merged 3 commits intomasterfrom
feature/schedulable-rewrite
Apr 4, 2026
Merged

Add modal field validation using Marmalade Validator#6
Aerhhh merged 3 commits intomasterfrom
feature/schedulable-rewrite

Conversation

@Aerhhh
Copy link
Copy Markdown
Contributor

@Aerhhh Aerhhh commented Apr 4, 2026

Summary

  • Added server-side validation of ticket modal fields using Marmalade Validator
  • All field errors (required, minLength, maxLength) shown to the user at once instead of one at a time
  • Defence-in-depth guard against clients bypassing Discord's native modal constraints

Aerhhh added 3 commits April 4, 2026 22:20
@Aerhhh
feat: validate all ticket modal fields at once using Marmalade Validator
e4b1622
@Aerhhh
fix: update TicketRepository.findById for Result return type
74229eb 
Override now returns Result<Ticket, RepositoryException> to match the
updated Repository<T> base class signature. Adds MongoException handling
for database errors and wraps cache hits and DB lookups in Result.success/failure.
Updates the internal findByChannelId caller to unwrap the Result correctly.
@Aerhhh
build: target Java 25
a5307aa
@Aerhhh Aerhhh merged commit 4a41179 into master Apr 4, 2026
1 of 2 checks passed
@Aerhhh Aerhhh deleted the feature/schedulable-rewrite branch April 4, 2026 21:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@Aerhhh