Skip to content

feat(web): add per-network betterstack url configuration#625

Open
silent-cipher wants to merge 4 commits into
refactor/web/multi-network-uifrom
feat/web/per-network-betterstack-urls
Open

feat(web): add per-network betterstack url configuration#625
silent-cipher wants to merge 4 commits into
refactor/web/multi-network-uifrom
feat/web/per-network-betterstack-urls

Conversation

@silent-cipher

Copy link
Copy Markdown
Collaborator

Built on top of #576

@FilOzzy FilOzzy added this to FOC Jun 29, 2026
@github-project-automation github-project-automation Bot moved this to 📌 Triage in FOC Jun 29, 2026
@silent-cipher silent-cipher requested a review from SgtPooki June 29, 2026 18:14
@silent-cipher silent-cipher self-assigned this Jun 29, 2026
@BigLep BigLep moved this from 📌 Triage to 🔎 Awaiting review in FOC Jun 29, 2026
@silent-cipher silent-cipher requested a review from Copilot July 1, 2026 18:06

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR extends the web app’s BetterStack configuration to support per-network URLs (mainnet vs calibration), building on the multi-network support introduced in #576. It adds new env/runtime config keys and updates the landing page config resolution logic to prefer network-scoped URLs with a global fallback for existing single-network deployments.

Changes:

  • Add per-network DASHBOARD_URL and LOGS_URL config keys (Vite env + runtime window.__DEALBOT_CONFIG__).
  • Update landing-page config resolution to prefer per-network URLs with a global fallback.
  • Document the new env vars in apps/web/.env.example and plumb them through the Docker runtime config generator.

Reviewed changes

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

File Description
apps/web/src/vite-env.d.ts Adds typings for per-network dashboard/logs env + runtime config keys.
apps/web/src/pages/Landing.tsx Introduces getNetworkConfigUrl and resolves dashboard/logs URLs per network with fallback.
apps/web/docker-entrypoint.sh Injects per-network dashboard/logs URLs into window.__DEALBOT_CONFIG__ at container start.
apps/web/.env.example Documents new per-network BetterStack URL variables and fallback behavior.

Comment thread apps/web/src/pages/Landing.tsx Outdated

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 4 out of 5 changed files in this pull request and generated 1 comment.

Comment thread apps/web/src/pages/Landing.tsx Outdated

@BigLep BigLep left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approving. Quick scan seems right and this is low risk to get wrong.

@github-project-automation github-project-automation Bot moved this from 🔎 Awaiting review to ✔️ Approved by reviewer in FOC Jul 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✔️ Approved by reviewer

Development

Successfully merging this pull request may close these issues.

4 participants