feat(issue-reporter): implement in-app issue reporter modal and Cloudflare API#225
Closed
ThisIs-Developer wants to merge 2 commits into
Closed
feat(issue-reporter): implement in-app issue reporter modal and Cloudflare API#225ThisIs-Developer wants to merge 2 commits into
ThisIs-Developer wants to merge 2 commits into
Conversation
…flare API - Replace direct GitHub new-issue link redirect with an in-app popup modal matching Share Snapshot / Live Share UI design. - Implement Cloudflare Pages Function at /api/report-issue for secure server-side GitHub Issue creation via GitHub REST API. - Support issue types, title, description, steps to reproduce, expected/actual behavior, drag-and-drop attachments (R2 storage), and environment metadata. - Implement rate limiting, origin validation, Turnstile support, client validation, double-click lock, and backward-compatible fallback mode. - Add setup documentation in docs/issue-reporter-setup.md.
Deploying markdownviewer with
|
| Latest commit: |
62de12e
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://f53e7a4d.markdownviewer.pages.dev |
| Branch Preview URL: | https://feat-in-app-issue-reporter.markdownviewer.pages.dev |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Prevent Cloudflare Pages publish failure when the R2 bucket 'markdown-viewer-attachments' is not pre-provisioned in the deployment environment. R2 bucket can be bound optionally in Cloudflare Pages settings.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changed
Why it changed
How it was tested
pm run build: Static build check passed.
px playwright test tests/e2e/report-issue.spec.js: 5 E2E tests passed (modal open/close, Escape key dismiss, field validation, mock successful submission, error & fallback button).
pm test: Full Playwright E2E test suite passed (91 tests).
Privacy and Storage Notes
Documentation Updates