Skip to content

fix: replace report.html GitHub issue builder with researchskills.ai#62

Draft
ktwu01 wants to merge 1 commit intomainfrom
fix/report-html-simplify-submit
Draft

fix: replace report.html GitHub issue builder with researchskills.ai#62
ktwu01 wants to merge 1 commit intomainfrom
fix/report-html-simplify-submit

Conversation

@ktwu01
Copy link
Copy Markdown
Collaborator

@ktwu01 ktwu01 commented Apr 20, 2026

Summary

  • Remove ~100 lines of buildIssueUrl() JS that constructed GitHub issue URLs with inline markdown YAML
  • Replace with simple redirect to researchskills.ai/submit
  • Update button text from "Submit to GitHub" → "Submit" and "Submit All Accepted to GitHub" → "Submit All Accepted"
  • Consent gating (consentGiven checks) preserved throughout

Supersedes #57 (rebased on current main with consent banner intact).

Test plan

  • Open generated report.html, click per-skill "Submit" button → should open researchskills.ai/submit
  • Click "Submit All Accepted" → should open researchskills.ai/submit with count message
  • Verify consent checkbox must be checked before either submit button works
  • Verify no JavaScript errors in browser console

🤖 Generated with Claude Code

…redirect

Remove ~100 lines of buildIssueUrl() JS that constructed GitHub issue
URLs with inline markdown YAML. Replace with simple redirect to
researchskills.ai/submit. Consent gating preserved.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@ktwu01 ktwu01 requested a review from HHHHHejia as a code owner April 20, 2026 18:11
@ktwu01 ktwu01 marked this pull request as draft April 20, 2026 18:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant