Skip to content

chore: remove my-site references#167

Merged
iamitprakash merged 1 commit intodevelopfrom
chore/remove-my-site
Jan 12, 2026
Merged

chore: remove my-site references#167
iamitprakash merged 1 commit intodevelopfrom
chore/remove-my-site

Conversation

@MayankBansal12
Copy link
Member

@MayankBansal12 MayankBansal12 commented Jan 7, 2026

Date: 07-01-26

Developer Name: @MayankBansal12


Issue Ticket Number

Description

  • remove my-site config reference
    (note: mySiteUrl wasn't being anywhere in repo)

Documentation Updated?

  • Yes
  • No

Under Feature Flag

  • Yes
  • No

Database Changes

  • Yes
  • No

Breaking Changes

  • Yes
  • No

Development Tested?

  • Yes
  • No

Screenshot

N/A, (unused config removed)

@coderabbitai
Copy link

coderabbitai bot commented Jan 7, 2026

Walkthrough

Removed the NEXT_PUBLIC_MY_SITE_URL environment variable from .env.sample and deleted the corresponding mySiteUrl field from the configuration schema and exported config object in src/config.ts. This eliminates a configuration entry across both environment and runtime layers.

Changes

Cohort / File(s) Summary
Environment Configuration
.env.sample
Removed public environment variable NEXT_PUBLIC_MY_SITE_URL declaration
Runtime Configuration
src/config.ts
Removed mySiteUrl field from Zod schema (configSchema) and exported config object; eliminates type-inferred property from configuration shape

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Poem

🐰 A site URL once lived here so bright,
But now it hops away out of sight,
The config grows lean, the schema so clean,
Less clutter to manage—a tidy routine! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title 'chore: remove my-site references' directly aligns with the main changeset objective of removing mySiteUrl configuration from .env.sample and src/config.ts.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check ✅ Passed The pull request description clearly explains the changes: removing unused my-site config references (NEXT_PUBLIC_MY_SITE_URL and mySiteUrl), which directly aligns with the file modifications shown.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@iamitprakash iamitprakash merged commit 5c6dd4e into develop Jan 12, 2026
3 checks passed
@iamitprakash iamitprakash deleted the chore/remove-my-site branch January 12, 2026 12:39
@MayankBansal12 MayankBansal12 mentioned this pull request Jan 16, 2026
10 tasks
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.

2 participants