Skip to content

chore(config): Update README title and env values #18#19

Merged
mathemage merged 7 commits into
mainfrom
docs/18-readme-title-env-values
Mar 29, 2026
Merged

chore(config): Update README title and env values #18#19
mathemage merged 7 commits into
mainfrom
docs/18-readme-title-env-values

Conversation

@mathemage
Copy link
Copy Markdown
Owner

@mathemage mathemage commented Mar 29, 2026

Summary

  • update the README title to the requested repository heading
  • move the public site URL, Instagram handle, and booking URL into tracked src/content/site.ts
  • keep .env.example focused on environment-specific settings and secrets
  • normalize the canonical site URL before deriving sitemap, robots, and metadata URLs

Validation

  • pnpm lint
  • pnpm test
  • pnpm build

Closes #18

mathemage and others added 2 commits March 29, 2026 22:29
Use the requested descriptive README heading for the repository landing page.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Record the requested SITE_URL, COACH_IG_USERNAME, and BOOKING_URL values in the tracked env template while keeping the matching .env.local update uncommitted.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 29, 2026

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

Project Deployment Actions Updated (UTC)
boulder-daddy Ready Ready Preview, Comment Mar 29, 2026 10:39pm

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates public-facing configuration documentation and defaults to reflect the “boulder-daddy” repository branding and intended deployment values.

Changes:

  • Rename the README heading to the requested project title/description.
  • Populate .env.example with public default values for SITE_URL, COACH_IG_USERNAME, and BOOKING_URL.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
README.md Updates the repository README title/heading.
.env.example Sets concrete public defaults for site URL, Instagram username, and booking URL.

Comment thread .env.example Outdated
Remove the trailing slash from SITE_URL in the tracked env template so generated sitemap and robots URLs do not contain double slashes.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Move the canonical site URL, Instagram handle, and booking URL into tracked site content so the repository stays the source of truth for public values. Update docs and consumers to read those values from src/content/site.ts instead of env vars.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 10 out of 10 changed files in this pull request and generated 3 comments.

Comment thread .env.example
Comment thread src/app/sitemap.ts Outdated
Comment thread src/app/robots.ts Outdated
@mathemage mathemage self-assigned this Mar 29, 2026
@mathemage mathemage added the enhancement New feature or request label Mar 29, 2026
Normalize the tracked site URL before sitemap, robots, and metadata consumers build derived URLs, and add a regression test for the helper.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 11 out of 11 changed files in this pull request and generated 1 comment.

Comment thread src/content/site.ts Outdated
Trim whitespace and strip one-or-more trailing slashes from the tracked site URL normalizer, and cover the edge case with a regression test.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 11 out of 11 changed files in this pull request and generated 2 comments.

Comment thread src/__tests__/site-config.test.ts Outdated
Comment thread README.md Outdated
Update the README to reflect the runtime default for INSTAGRAM_MODE and tighten the site URL normalization test to compare against the helper output.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 11 out of 11 changed files in this pull request and generated no new comments.

@mathemage mathemage merged commit 0742bf4 into main Mar 29, 2026
7 checks passed
@mathemage mathemage deleted the docs/18-readme-title-env-values branch March 29, 2026 22:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update README title and remaining environment values

2 participants