Skip to content

Docs#44

Merged
alexgarrettsmith merged 4 commits into
mainfrom
docs
Jul 8, 2026
Merged

Docs#44
alexgarrettsmith merged 4 commits into
mainfrom
docs

Conversation

@alexgarrettsmith

Copy link
Copy Markdown
Collaborator

No description provided.

alexgarrettsmith and others added 4 commits July 8, 2026 09:07
Create one docs page per feature under /docs, covering authentication,
account settings, feedback, voting, reactions, comments, notifications,
the internal dashboard, public pages, and the changelog workflow.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Shared git hook (.githooks, enabled via core.hooksPath) runs the
docs-writer subagent headlessly on the staged diff at commit time and
auto-stages any /docs updates. Un-ignore .claude/agents so the subagent
is version-controlled while local .claude settings stay ignored.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Medium Risk

Docs Automation Tooling
Reason: A shell pre-commit hook invokes the external claude CLI with acceptEdits permissions and complex bash conditionals to auto-write files, affecting every developer's commit flow.

  • .githooks/pre-commit (added)
  • .githooks/README.md (added)
  • .claude/agents/docs-writer.md (added)
  • .gitignore (modified)

Low Risk

Reactions

  • app/Models/Reaction.php (modified)
  • docs/reactions.md (added)

Project Documentation

  • docs/README.md (added)
  • docs/account-settings.md (added)
  • docs/authentication.md (added)
  • docs/changelog.md (added)
  • docs/comments.md (added)
  • docs/feedback.md (added)
  • docs/internal-dashboard.md (added)
  • docs/notifications.md (added)
  • docs/public-pages.md (added)
  • docs/voting.md (added)

3 groups, 0 high-risk files to focus on

@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

QA checklist — Add 😂 and 🔥 to the available reaction emoji

Before you start: Sign in with a test account and open an existing feedback idea's detail page. Have the app deployed with the new build.

  • On the idea page, click the smiley/"Add a reaction" button to open the reaction picker — the popover should list seven emoji, including the two new ones (😂 and 🔥) alongside 👍 ❤️ 🎉 🚀 👀.
  • In the picker, click 🔥 — the popover closes, the page stays scrolled in place, and a 🔥 chip appears showing a count of 1 and highlighted as your reaction.
  • Click 😂 from the picker — a 😂 chip appears with count 1, and your 🔥 chip is still present (you can hold several different reactions on one idea).
  • Click your highlighted 🔥 chip again — the reaction toggles off; its count drops (and the chip disappears if you were the only reactor).
  • React with one of the original emoji (e.g. 👍) to confirm existing reactions still work unchanged — the 👍 chip appears with the correct count.
  • Reload the idea page — your remaining reactions persist with correct counts and stay highlighted for your account.
  • Sign out (or open the idea as a guest) — the reaction button is disabled with a "Sign in to react" tooltip and existing reaction chips are not clickable.

@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Configuration
─────────────

Required

  • Activate the shared git hooks in each clone with git config core.hooksPath .githooks, otherwise the docs pre-commit hook never runs — .githooks/README.md

Verify

  • Ensure the claude CLI is installed and authenticated wherever commits are made, so the docs-writer hook can invoke claude -p against the staged diff — .githooks/pre-commit
  • Confirm the docs-writer subagent file ships to other clones; the .gitignore change now un-ignores /.claude/agents/ specifically so the hook's subagent is available — .gitignore

Operational
───────────

Nice to confirm

  • Check that the two new reaction emoji (😂 🔥) render correctly on the idea page — app/Models/Reaction.php
  • Note the docs pre-commit hook is best-effort: it is skipped and the commit still proceeds if claude is missing or the run fails, so it will not block deploys/commits — .githooks/pre-commit

@alexgarrettsmith alexgarrettsmith merged commit 0dc9359 into main Jul 8, 2026
4 checks passed
@alexgarrettsmith alexgarrettsmith deleted the docs branch July 8, 2026 08:24
alexgarrettsmith added a commit that referenced this pull request Jul 8, 2026
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