Skip to content

Open-source launch docs and acceptance test#4

Closed
bnz183 wants to merge 1 commit into
split/02-github-studio-polishfrom
split/03-launch-docs
Closed

Open-source launch docs and acceptance test#4
bnz183 wants to merge 1 commit into
split/02-github-studio-polishfrom
split/03-launch-docs

Conversation

@bnz183

@bnz183 bnz183 commented Jun 9, 2026

Copy link
Copy Markdown
Owner

PR 3 of 11. Issue templates, changelog, screenshots docs, manual acceptance test.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: bfe48f1b9e

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".


- [ ] `pnpm install`
- [ ] Copy `sourcedraft.config.example.json` → `sourcedraft.config.json` and adjust paths
- [ ] Copy `.env.example` → `.env` with `CMS_ADMIN_PASSWORD`, `GITHUB_TOKEN`, `GITHUB_OWNER`, `GITHUB_REPO`

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Use the actual admin password environment variable

Following this setup step leaves Studio auth unconfigured because the server reads only SOURCEDRAFT_ADMIN_PASSWORD (apps/studio/server/auth.ts) and .env.example defines that same name, not CMS_ADMIN_PASSWORD. In the manual acceptance flow this causes the subsequent login step to fail with “SOURCEDRAFT_ADMIN_PASSWORD is not configured” even though the checklist was followed.

Useful? React with 👍 / 👎.

@bnz183

bnz183 commented Jun 9, 2026

Copy link
Copy Markdown
Owner Author

Closing retroactive split-stack review PR. Continuing from protected main with scoped feature PRs.

@bnz183 bnz183 closed this Jun 9, 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