Skip to content

OUT-4155: condition check in config for staging environment as prod use local now - #284

Merged
SandipBajracharya merged 1 commit into
masterfrom
OUT-4155
Sep 4, 2026
Merged

OUT-4155: condition check in config for staging environment as prod use local now#284
SandipBajracharya merged 1 commit into
masterfrom
OUT-4155

Conversation

@SandipBajracharya

Copy link
Copy Markdown
Collaborator

Changes

  • condition update to check for staging environment for domain url

Testing Criteria

  • Screenshot after implementation
image

@linear-code

linear-code Bot commented Sep 4, 2026

Copy link
Copy Markdown

OUT-4155

@vercel

vercel Bot commented Sep 4, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated
quickbooks-sync Ready Ready Preview Sep 4, 2026 5:51am UTC

Request Review

@greptile-apps

greptile-apps Bot commented Sep 4, 2026

Copy link
Copy Markdown

Greptile Summary

This PR changes domain selection so only the designated staging environment uses staging dashboard and API URLs; all other environments use production URLs.

  • Reverses the previous production-only condition into a staging-only condition.
  • Applies the same environment selection consistently to dashboard origins and the Copilot API domain.

Confidence Score: 5/5

The PR appears safe to merge, with no concrete changed-code failure established.

The dashboard and API domain constants consistently implement the stated staging-only exception, and the repository provides no evidence that a reachable deployment uses an incompatible staging environment value.

Important Files Changed

Filename Overview
src/constant/domains.ts Updates both domain constants to select staging URLs only for the designated staging environment, with no concrete defect established.

Reviews (1): Last reviewed commit: "fix(OUT-4155): condition check in config..." | Re-trigger Greptile

@priosshrsth priosshrsth left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lgtm

@SandipBajracharya
SandipBajracharya merged commit c892a23 into master Sep 4, 2026
6 checks passed
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