Fixed by PR #126 (#126): removed the '(members, teams, invitations — 404 unless this owner is an org)' footer prose from web/src/pages/Repos.jsx. Both links (import into , organization settings) stay, so nothing is stranded.

Fixed by PR #126 (https://git.packden.us/crueber/walhub/pulls/126): removed the '(members, teams, invitations — 404 unless this owner is an org)' footer prose from web/src/pages/Repos.jsx. Both links (import into <owner>, organization settings) stay, so nothing is stranded.

crueber commented

2026-09-05 05:49:58 +00:00

Author

Owner

Review of PR #126 (fix/issue-116, commit 639286e):

Scope: diff is exactly one deleted line in web/src/pages/Repos.jsx (~line 50): the hint prose ' (members, teams, invitations — 404 unless this owner is an org)'. Both links (import into {owner}, organization settings) are intact and untouched. Nothing else in the diff; no package.json/go.mod changes (law 1 clean, no new deps).

Laws spot-check: 1 pass (no deps), 7 N/A (static text, no async work), 8 pass (no new capability/registry touch), 12 pass — grepped docs/ for 'members, teams, invitations': no hits, so no doc pinned the text and no doc update needed. Only other hit repo-wide is the gitignored local build artifact web/dist/assets/*.js (not tracked, not part of the PR).

Verify (scratch worktree /tmp/opencode/fix-116 @ 639286e, main worktree untouched read-only): node --test web/test/unit/*.test.js → 257 pass, 0 fail (initial 2 failures were missing node_modules in the fresh worktree — env-only, resolved via temp symlink to main's node_modules, symlink removed after); vite build → success (110 modules, built in ~2s). No browser drive per task instructions (text-only deletion; noted, not run).

No fixes needed. MERGE RECOMMENDATION: ready to merge.

Review of PR #126 (fix/issue-116, commit 639286e): Scope: diff is exactly one deleted line in web/src/pages/Repos.jsx (~line 50): the hint prose ' (members, teams, invitations — 404 unless this owner is an org)'. Both links (import into {owner}, organization settings) are intact and untouched. Nothing else in the diff; no package.json/go.mod changes (law 1 clean, no new deps). Laws spot-check: 1 pass (no deps), 7 N/A (static text, no async work), 8 pass (no new capability/registry touch), 12 pass — grepped docs/ for 'members, teams, invitations': no hits, so no doc pinned the text and no doc update needed. Only other hit repo-wide is the gitignored local build artifact web/dist/assets/*.js (not tracked, not part of the PR). Verify (scratch worktree /tmp/opencode/fix-116 @ 639286e, main worktree untouched read-only): node --test web/test/unit/*.test.js → 257 pass, 0 fail (initial 2 failures were missing node_modules in the fresh worktree — env-only, resolved via temp symlink to main's node_modules, symlink removed after); vite build → success (110 modules, built in ~2s). No browser drive per task instructions (text-only deletion; noted, not run). No fixes needed. MERGE RECOMMENDATION: ready to merge.

crueber closed this issue

2026-09-05 05:50:26 +00:00

crueber referenced this issue from a commit

2026-09-05 05:50:26 +00:00

Merge pull request 'Fix #116: drop org page hint text' (#126) from fix/issue-116 into main

crueber commented

2026-09-05 05:50:26 +00:00

Author

Owner

Fixed by PR #126 (review clean; 257/257 node tests), merged. Closing.

crueber added the

git storage