Skip to content

improvement(docs): collect reader feedback inline - #9538

Merged
waleedlatif1 merged 3 commits into
stagingfrom
codex/docs-polish-feedback
Oct 31, 2026
Merged

waleedlatif1 merged 3 commits into
stagingfrom
codex/docs-polish-feedback

Conversation

@waleedlatif1

Copy link
Copy Markdown
Collaborator

Summary

  • Collect documentation votes and optional comments inline through an authenticated server-side workflow intake, with bounded validation and retry-safe submission identities.
  • Preserve GitHub edit links and redirect /docs and its deep links to the current documentation site using temporary redirects.

Type of Change

  • Improvement

Testing

  • Focused Biome 2.0.6 and diff checks passed; parallel cleanup reviews completed.
  • Verified authenticated workflow intake, stored feedback, and duplicate submission behavior.
  • Added HTTP rejection coverage to the existing docs interface suite. Full builds, tests, audits, and deployed UI QA are pending CI.

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing (new tests pass the test-audit authoring gate)
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel

vercel Bot commented Oct 31, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
docs Ready Ready Preview Oct 31, 2026 6:59pm UTC

Request Review

@greptile-apps

greptile-apps Bot commented Oct 31, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 5/5

[Medium impact] The PR appears safe to merge; no actionable defect was found.

Summary

Adds inline documentation votes and optional comments, sent through a server-side Sim webhook.

  • Docs readers can vote and leave optional comments inline.
  • Old /docs links now redirect to the current documentation site.

The added navigation scenario addresses the earlier, unnumbered testing comment. This review checked the code; it did not run the suites or capture the UI because the workspace has no installed Next.js package or production build.

Diagram

sequenceDiagram
    participant Reader
    participant Form as Feedback form
    participant Docs as Docs server
    participant Sim as Sim webhook
    participant Job as Workflow job
    Reader->>Form: Choose a vote and optional comment
    Form->>Docs: POST /api/feedback with submission ID
    Docs->>Docs: Check origin, size, fields, and page
    Docs->>Sim: HTTPS POST with server-held token
    Sim->>Job: Queue feedback
    Sim-->>Docs: Webhook processed
    Docs-->>Form: 202 accepted
    Form-->>Reader: Show thanks
    Job->>Job: Check submission ID before running
Loading

Reviews (3) · Last reviewed commit: "fix(docs): normalize feedback paths and ..." · Reviewed by Greptile

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All reported issues were addressed across 7 files

Reply with feedback, questions, or to request a fix.

Turn on auto-fix | Re-trigger cubic

Comment thread apps/docs/app/api/feedback/route.ts Resolved
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 31, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All reported issues were addressed across 9 files

Reply with feedback, questions, or to request a fix.

Turn on auto-fix | Re-trigger cubic

Comment thread apps/docs/app/api/feedback/route.ts Resolved
@waleedlatif1
waleedlatif1 force-pushed the codex/docs-polish-feedback branch from ee5cdeb to c9a3eb5 Compare October 31, 2026 18:54
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 31, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All reported issues were addressed across 9 files

Reply with feedback, questions, or to request a fix.

Turn on auto-fix | Re-trigger cubic

Comment thread apps/docs/app/api/feedback/route.ts Resolved

This branch was successfully deployed

1 active deployment
Preview — c9a3eb50 Deployed Oct 31, 2026 by vercel[bot]
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