Compare commits
1 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| 6bb53889da |
@@ -38,7 +38,6 @@ func (h *Handler) RegisterRoutes(mux *http.ServeMux) {
|
|||||||
|
|
||||||
// Issues.
|
// Issues.
|
||||||
mux.HandleFunc("GET /issues", h.ListIssues)
|
mux.HandleFunc("GET /issues", h.ListIssues)
|
||||||
mux.HandleFunc("GET /issues/new", h.NewIssue)
|
|
||||||
mux.HandleFunc("POST /issues", h.CreateIssue)
|
mux.HandleFunc("POST /issues", h.CreateIssue)
|
||||||
mux.HandleFunc("POST /issues/{owner}/{repo}/{index}/labels", h.ApplyLabels)
|
mux.HandleFunc("POST /issues/{owner}/{repo}/{index}/labels", h.ApplyLabels)
|
||||||
mux.HandleFunc("POST /issues/{owner}/{repo}/{index}/close", h.CloseIssue)
|
mux.HandleFunc("POST /issues/{owner}/{repo}/{index}/close", h.CloseIssue)
|
||||||
@@ -470,41 +469,6 @@ func (h *Handler) PullDetail(w http.ResponseWriter, r *http.Request) {
|
|||||||
renderPage(w, r, fmt.Sprintf("PR #%d", index), "pulls", buf.String())
|
renderPage(w, r, fmt.Sprintf("PR #%d", index), "pulls", buf.String())
|
||||||
}
|
}
|
||||||
|
|
||||||
// NewIssue handles GET /issues/new — renders the create-issue form.
|
|
||||||
func (h *Handler) NewIssue(w http.ResponseWriter, r *http.Request) {
|
|
||||||
token := getToken(r)
|
|
||||||
|
|
||||||
repos, err := h.Client.ListOrgsAndRepos(r.Context(), token)
|
|
||||||
if err != nil {
|
|
||||||
slog.Error("failed to list repos for new issue form", "error", err)
|
|
||||||
renderPage(w, r, "New Issue", "issues",
|
|
||||||
`<h1>New Issue</h1><p class="empty">Error loading repositories.</p>`)
|
|
||||||
return
|
|
||||||
}
|
|
||||||
|
|
||||||
tmpl, err := template.ParseFiles("internal/templates/create_issue.html")
|
|
||||||
if err != nil {
|
|
||||||
slog.Error("failed to parse create_issue template", "error", err)
|
|
||||||
http.Error(w, "template error", http.StatusInternalServerError)
|
|
||||||
return
|
|
||||||
}
|
|
||||||
|
|
||||||
type templateData struct {
|
|
||||||
Repos map[string][]giteaclient.Repo
|
|
||||||
}
|
|
||||||
|
|
||||||
data := templateData{Repos: repos}
|
|
||||||
|
|
||||||
var buf strings.Builder
|
|
||||||
if err := tmpl.ExecuteTemplate(&buf, "content", data); err != nil {
|
|
||||||
slog.Error("failed to execute create_issue template", "error", err)
|
|
||||||
http.Error(w, "template error", http.StatusInternalServerError)
|
|
||||||
return
|
|
||||||
}
|
|
||||||
|
|
||||||
renderPage(w, r, "New Issue", "issues", buf.String())
|
|
||||||
}
|
|
||||||
|
|
||||||
// CreateIssue handles POST /issues.
|
// CreateIssue handles POST /issues.
|
||||||
func (h *Handler) CreateIssue(w http.ResponseWriter, r *http.Request) {
|
func (h *Handler) CreateIssue(w http.ResponseWriter, r *http.Request) {
|
||||||
token := getToken(r)
|
token := getToken(r)
|
||||||
@@ -519,12 +483,6 @@ func (h *Handler) CreateIssue(w http.ResponseWriter, r *http.Request) {
|
|||||||
body := r.FormValue("body")
|
body := r.FormValue("body")
|
||||||
|
|
||||||
if owner == "" || repo == "" || title == "" {
|
if owner == "" || repo == "" || title == "" {
|
||||||
if isHTMX(r) {
|
|
||||||
w.Header().Set("Content-Type", "text/html; charset=utf-8")
|
|
||||||
w.WriteHeader(http.StatusBadRequest)
|
|
||||||
fmt.Fprint(w, `<span class="empty">owner, repo, and title are required</span>`)
|
|
||||||
return
|
|
||||||
}
|
|
||||||
http.Error(w, "owner, repo, and title are required", http.StatusBadRequest)
|
http.Error(w, "owner, repo, and title are required", http.StatusBadRequest)
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
@@ -532,12 +490,6 @@ func (h *Handler) CreateIssue(w http.ResponseWriter, r *http.Request) {
|
|||||||
issue, err := h.Client.CreateIssue(r.Context(), token, owner, repo, title, body, nil)
|
issue, err := h.Client.CreateIssue(r.Context(), token, owner, repo, title, body, nil)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
slog.Error("failed to create issue", "error", err)
|
slog.Error("failed to create issue", "error", err)
|
||||||
if isHTMX(r) {
|
|
||||||
w.Header().Set("Content-Type", "text/html; charset=utf-8")
|
|
||||||
w.WriteHeader(http.StatusInternalServerError)
|
|
||||||
fmt.Fprint(w, `<span class="empty">Failed to create issue. Please try again.</span>`)
|
|
||||||
return
|
|
||||||
}
|
|
||||||
http.Error(w, "failed to create issue", http.StatusInternalServerError)
|
http.Error(w, "failed to create issue", http.StatusInternalServerError)
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -1,9 +1,7 @@
|
|||||||
{{define "content"}}
|
{{define "content"}}
|
||||||
<h1>Create Issue</h1>
|
<h1>Create Issue</h1>
|
||||||
|
|
||||||
<div id="form-error" class="empty" style="display:none;"></div>
|
<form hx-post="/issues" hx-swap="innerHTML" hx-target="#main-content">
|
||||||
|
|
||||||
<form id="create-issue-form" hx-post="/issues" hx-swap="innerHTML" hx-target="#main-content">
|
|
||||||
<div class="form-group">
|
<div class="form-group">
|
||||||
<label for="repo-select">Repository</label>
|
<label for="repo-select">Repository</label>
|
||||||
<select id="repo-select" name="owner_repo" required>
|
<select id="repo-select" name="owner_repo" required>
|
||||||
@@ -34,43 +32,11 @@
|
|||||||
</form>
|
</form>
|
||||||
|
|
||||||
<script>
|
<script>
|
||||||
(function() {
|
// Split owner/repo from select into hidden fields.
|
||||||
var repoSelect = document.getElementById('repo-select');
|
document.getElementById('repo-select').addEventListener('change', function() {
|
||||||
var ownerInput = document.getElementById('owner-input');
|
var parts = this.value.split('/');
|
||||||
var repoInput = document.getElementById('repo-input');
|
document.getElementById('owner-input').value = parts[0] || '';
|
||||||
var formError = document.getElementById('form-error');
|
document.getElementById('repo-input').value = parts[1] || '';
|
||||||
|
});
|
||||||
function splitOwnerRepo() {
|
|
||||||
var val = repoSelect.value;
|
|
||||||
if (val) {
|
|
||||||
var parts = val.split('/');
|
|
||||||
ownerInput.value = parts[0] || '';
|
|
||||||
repoInput.value = parts[1] || '';
|
|
||||||
} else {
|
|
||||||
ownerInput.value = '';
|
|
||||||
repoInput.value = '';
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
repoSelect.addEventListener('change', splitOwnerRepo);
|
|
||||||
|
|
||||||
// Validate before HTMX submit.
|
|
||||||
document.getElementById('create-issue-form').addEventListener('htmx:configRequest', function(evt) {
|
|
||||||
splitOwnerRepo();
|
|
||||||
if (!ownerInput.value || !repoInput.value) {
|
|
||||||
evt.preventDefault();
|
|
||||||
formError.textContent = 'Please select a repository before submitting.';
|
|
||||||
formError.style.display = 'block';
|
|
||||||
return false;
|
|
||||||
}
|
|
||||||
formError.style.display = 'none';
|
|
||||||
});
|
|
||||||
|
|
||||||
// Show server-side errors inline on HTMX error responses.
|
|
||||||
document.getElementById('create-issue-form').addEventListener('htmx:responseError', function(evt) {
|
|
||||||
formError.textContent = evt.detail.xhr.responseText || 'An error occurred. Please try again.';
|
|
||||||
formError.style.display = 'block';
|
|
||||||
});
|
|
||||||
})();
|
|
||||||
</script>
|
</script>
|
||||||
{{end}}
|
{{end}}
|
||||||
|
|||||||
Reference in New Issue
Block a user