Skip to content

chore(analytics): updates staging proxy url for amplitude#2655

Merged
ygrishajev merged 2 commits intomainfrom
chore/analytics
Feb 3, 2026
Merged

chore(analytics): updates staging proxy url for amplitude#2655
ygrishajev merged 2 commits intomainfrom
chore/analytics

Conversation

@ygrishajev
Copy link
Contributor

@ygrishajev ygrishajev commented Feb 3, 2026

Summary by CodeRabbit

  • Chores
    • Reorganized environment configuration by moving settings from base configuration to environment-specific files for production and staging deployments.

@ygrishajev ygrishajev requested a review from a team as a code owner February 3, 2026 09:52
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 3, 2026

📝 Walkthrough

Walkthrough

Environment configuration for the Amplitude proxy URL is being migrated from the shared .env file to environment-specific files, with production and staging instances using different endpoints (console-proxy and console-proxy-beta subdomains respectively).

Changes

Cohort / File(s) Summary
Amplitude Proxy Configuration
apps/deploy-web/env/.env, apps/deploy-web/env/.env.production, apps/deploy-web/env/.env.staging
Removed NEXT_PUBLIC_AMPLITUDE_PROXY_URL from base .env and added environment-specific values to .env.production (console-proxy.akash.network) and .env.staging (console-proxy-beta.akash.network).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • stalniy
  • baktun14

Poem

🐰 The proxy hops from .env to staging's door,
With production paths that route evermore,
Each subdomain tweaked with hop and bound,
Analytics proxies safely configured round! 🎯

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the main change: updating the staging proxy URL for Amplitude analytics. However, the changes also include removing the variable from .env and adding it to .env.production, which are not mentioned in the title.
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.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/analytics

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

@codecov
Copy link

codecov bot commented Feb 3, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 50.60%. Comparing base (823030c) to head (7cae3f3).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2655      +/-   ##
==========================================
- Coverage   51.41%   50.60%   -0.81%     
==========================================
  Files        1050     1018      -32     
  Lines       29531    28750     -781     
  Branches     6634     6538      -96     
==========================================
- Hits        15184    14550     -634     
+ Misses      14064    13919     -145     
+ Partials      283      281       -2     
Flag Coverage Δ *Carryforward flag
api 78.31% <ø> (-0.33%) ⬇️ Carriedforward from f945c14
deploy-web 32.75% <ø> (ø)
log-collector ?
notifications 87.94% <ø> (ø) Carriedforward from f945c14
provider-console 81.48% <ø> (ø) Carriedforward from f945c14
provider-proxy 84.35% <ø> (ø) Carriedforward from f945c14
tx-signer ?

*This pull request uses carry forward flags. Click here to find out more.
see 46 files with indirect coverage changes

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ygrishajev ygrishajev merged commit 0ddf032 into main Feb 3, 2026
52 checks passed
@ygrishajev ygrishajev deleted the chore/analytics branch February 3, 2026 13:06
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

Comments