Skip to content

chore: replace my site url for verification#314

Merged
iamitprakash merged 2 commits intodevelopfrom
fix/replace-my-site
Jan 14, 2026
Merged

chore: replace my site url for verification#314
iamitprakash merged 2 commits intodevelopfrom
fix/replace-my-site

Conversation

@MayankBansal12
Copy link
Member

@MayankBansal12 MayankBansal12 commented Jan 8, 2026

Date: 08-01-26

Developer Name: @MayankBansal12


Issue Ticket Number

Description

  • Replace my site url with main site for verification
  • Update contribution.md to replace my site

Documentation Updated?

  • Yes
  • No

Under Feature Flag

  • Yes
  • No

Database Changes

  • Yes
  • No

Breaking Changes

  • Yes
  • No

Development Tested?

  • Yes
  • No

Screenshots

proof for local development, localhost:4200 is being used image

@coderabbitai
Copy link

coderabbitai bot commented Jan 8, 2026

Important

Review skipped

Auto incremental reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Walkthrough

This pull request updates URL references and documentation text from a "my" subdomain/endpoint to a "www" subdomain/endpoint. Changes span configuration constants, contribution documentation, and user-facing messages, reflecting a consistent migration to new hostnames and ports across three environments (production, development, staging).

Changes

Cohort / File(s) Summary
URL Configuration
src/constants/urls.ts
Updated three URL constants: VERIFICATION_SITE_URL (my.realdevsquad.com → www.realdevsquad.com), DEVELOPMENT_VERIFICATION_SITE_URL (localhost:3443 → localhost:4200), STAGING_VERIFICATION_SITE_URL (staging-my.realdevsquad.com → staging-www.realdevsquad.com)
Documentation
CONTRIBUTING.md
Updated Discord contribution instruction reference from website-my repository to website-www repository
User-Facing Messages
src/utils/sendProfileServiceBlockedMessage.ts
Updated blocked profile message text to reference new main site URL instead of previous my site endpoint

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 From "my" to "www" we hop along,
The URLs dance to a newer song!
Constants updated, docs shining bright,
Messages redirected to the light,
Our warren's paths now point just right! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and directly summarizes the main change: replacing 'my site' URL with verification URLs across the codebase.
Description check ✅ Passed The description is directly related to the changeset, providing context about replacing my site URLs with main site URLs and updating documentation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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


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.

@MayankBansal12 MayankBansal12 marked this pull request as ready for review January 9, 2026 21:38
Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 2

🤖 Fix all issues with AI agents
In @src/utils/sendProfileServiceBlockedMessage.ts:
- Line 14: Replace the hardcoded "https://www.realdevsquad.com" with the
environment-aware VERIFICATION_SITE_URL from the config; import config from
"../../config/config" at the top of sendProfileServiceBlockedMessage.ts and
update the template string (the ternary branch that currently builds the message
containing [MAIN SITE](https://www.realdevsquad.com/identity)) to use
`${config.VERIFICATION_SITE_URL}/identity` so the link respects staging/dev/prod
environments.
📜 Review details

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 60e6cae and 9e421c6.

📒 Files selected for processing (3)
  • CONTRIBUTING.md
  • src/constants/urls.ts
  • src/utils/sendProfileServiceBlockedMessage.ts
🧰 Additional context used
🪛 markdownlint-cli2 (0.18.1)
CONTRIBUTING.md

137-137: Reversed link syntax
(website-www)[https://github.com/Real-Dev-Squad/website-www]

(MD011, no-reversed-links)

🔇 Additional comments (1)
src/constants/urls.ts (1)

9-12: URL constants are properly integrated across all environments.

The verification site URL constants are correctly mapped through the config system to their respective environments:

  • Production uses VERIFICATION_SITE_URL
  • Staging uses STAGING_VERIFICATION_SITE_URL
  • Development uses DEVELOPMENT_VERIFICATION_SITE_URL (port 4200)

All usages in src/controllers/verifyCommand.ts and tests access these constants through config(env), ensuring environment-specific URLs are applied correctly. No compatibility issues found.

@iamitprakash iamitprakash merged commit ceedefc into develop Jan 14, 2026
3 checks passed
@iamitprakash iamitprakash deleted the fix/replace-my-site branch January 14, 2026 15:27
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.

3 participants