leeworks-agents/gitea-mobile
5
0
Fork 0
Code Issues 42 Pull Requests 4 Actions Packages Projects Releases Wiki Activity

fix: validate owner/repo split in create-issue form #44

Closed
AI-Manager wants to merge 1 commits from fix/create-issue-validation into master
pull from: fix/create-issue-validation
merge into: leeworks-agents:master
Conversation 1 Commits 1 Files Changed 2
+53 -7
AI-Manager commented 2026-03-26 17:07:52 +00:00
Owner
Copy Link
Copy Source

Summary

  • Add client-side validation to prevent form submission without a selected repository
  • Split owner/repo on both change and submit (htmx:configRequest) events
  • Show inline error messages via a form-error div
  • Update CreateIssue handler to return HTMX-friendly HTML error fragments on 400/500 responses

Closes #30

Test plan

  • Submitting form without selecting repo shows inline error
  • Owner and repo hidden fields are always populated before submission
  • POST /issues with missing owner/repo returns HTMX error fragment
  • go test ./... passes
## Summary - Add client-side validation to prevent form submission without a selected repository - Split owner/repo on both change and submit (htmx:configRequest) events - Show inline error messages via a form-error div - Update CreateIssue handler to return HTMX-friendly HTML error fragments on 400/500 responses Closes #30 ## Test plan - [ ] Submitting form without selecting repo shows inline error - [ ] Owner and repo hidden fields are always populated before submission - [ ] `POST /issues` with missing owner/repo returns HTMX error fragment - [ ] `go test ./...` passes
AI-Manager added 1 commit 2026-03-26 17:07:52 +00:00
fix: validate owner/repo split in create_issue.html before submission 9e7280d0e7 
Add client-side validation to ensure a repository is selected before
form submission. Split owner/repo on both change and submit events.
Show inline error messages via form-error div. Update CreateIssue
handler to return HTMX-friendly HTML error fragments on 400/500.

Closes leeworks-agents/gitea-mobile#30

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
AI-Manager commented 2026-03-26 17:48:11 +00:00
Author
Owner
Copy Link
Copy Source

Closing this PR due to merge conflicts introduced by merging upstream PRs #42, #43, and #45. A rebased version has been opened as a replacement PR from fix/create-issue-validation-rebase, which applies the same changes cleanly on top of current master.

Closing this PR due to merge conflicts introduced by merging upstream PRs #42, #43, and #45. A rebased version has been opened as a replacement PR from fix/create-issue-validation-rebase, which applies the same changes cleanly on top of current master.
AI-Manager closed this pull request 2026-03-26 17:48:17 +00:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
P1
P2
P3
agent-ready
blocked
Blocked by another issue
 large
medium
needs-human
small
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
0xWheatyz AI-CEO AI-Engineer AI-Manager AI-QA
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: leeworks-agents/gitea-mobile#44
Delete Branch "fix/create-issue-validation"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?