Skip to content

fix: consolidate sprint 1 QA findings - #40

Merged
LoadCG merged 1 commit into
mainfrom
codex/sprint1-qa
Sep 27, 2026
Merged

LoadCG merged 1 commit into
mainfrom
codex/sprint1-qa

Conversation

@LoadCG

@LoadCG LoadCG commented Sep 27, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Reject seed targets unless the database name is explicitly dedicated with a _dev or _test suffix.
  • Build backend scripts in the seed validation CI job and document the local build prerequisite.
  • Run browser E2E regression on relevant pull requests and main updates.
  • Improve brand-link accessibility and restrict admin navigation/direct access to admin users.
  • Allow an environment-based Vite API proxy target for isolated environments and fix an unused auth prop that blocked TypeScript builds.

Verification

  • Backend: 318 tests passed with PostgreSQL integration tests enabled.
  • Frontend: 195 tests passed; production build passed.
  • Browser E2E: 22 tests passed in Chrome, including accessibility and mobile checks.
  • Backend build, typecheck, seed validation, seed tests, dependency audit, and git diff --check passed.

@LoadCG
LoadCG force-pushed the codex/sprint1-qa branch 2 times, most recently from be58f83 to d4b5a6a Compare September 27, 2026 18:58
@LoadCG
LoadCG merged commit d51a9e5 into main Sep 27, 2026
7 checks passed
@LoadCG
LoadCG deleted the codex/sprint1-qa branch September 27, 2026 19:07
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