Skip to content

[Dashboard] feat: use shared logger and cache headers - #3495

Merged
dnechay merged 2 commits into
developfrom
dnechay/fs-06082025
Aug 7, 2025
Merged

[Dashboard] feat: use shared logger and cache headers#3495
dnechay merged 2 commits into
developfrom
dnechay/fs-06082025

Conversation

@dnechay

@dnechay dnechay commented Aug 6, 2025

Copy link
Copy Markdown
Collaborator

Issue tracking

Follow up to #3387

Context behind the change

Added shared logger. Had to refactor files downloading logic in order to properly log errors coming from it.
Also added cache headers to responses where applicable in order to avoid unnecessary requests.

How has this been tested?

  • run Dashboard client & server locally; make sure all data is displayed correctly

Release plan

  • add NODE_ENV=production variable for corresponding service on Render (already done for stg)
  • enable logs sending from dashboard server -> DataDog on Render

Potential risks; What to monitor; Rollback plan

Check that logs are displayed correctly in DD.

@vercel

vercel Bot commented Aug 6, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

5 Skipped Deployments
Name Status Preview Comments Updated (UTC)
faucet-frontend ⬜️ Ignored (Inspect) Aug 6, 2025 3:03pm
faucet-server ⬜️ Ignored (Inspect) Visit Preview Aug 6, 2025 3:03pm
human-app ⬜️ Skipped (Inspect) Aug 6, 2025 3:03pm
human-dashboard-frontend ⬜️ Skipped (Inspect) Aug 6, 2025 3:03pm
staking-dashboard ⬜️ Skipped (Inspect) Aug 6, 2025 3:03pm

@dnechay
dnechay force-pushed the dnechay/fs-06082025 branch from 33b73cf to 522908a Compare August 6, 2025 12:58
@vercel
vercel Bot temporarily deployed to Preview – staking-dashboard August 6, 2025 12:59 Inactive
@vercel
vercel Bot temporarily deployed to Preview – human-app August 6, 2025 12:59 Inactive
@vercel
vercel Bot temporarily deployed to Preview – human-dashboard-frontend August 6, 2025 12:59 Inactive
@dnechay dnechay self-assigned this Aug 6, 2025
@dnechay dnechay moved this to In Progress in Core-tech - 2025 Aug 6, 2025
@dnechay dnechay moved this from In Progress to In Review in Core-tech - 2025 Aug 6, 2025
@dnechay
dnechay requested a review from Dzeranov August 6, 2025 13:04
@dnechay
dnechay force-pushed the dnechay/fs-06082025 branch from 522908a to 0fefe24 Compare August 6, 2025 15:00
@vercel
vercel Bot temporarily deployed to Preview – human-app August 6, 2025 15:03 Inactive
@vercel
vercel Bot temporarily deployed to Preview – human-dashboard-frontend August 6, 2025 15:03 Inactive
@vercel
vercel Bot temporarily deployed to Preview – staking-dashboard August 6, 2025 15:03 Inactive
@dnechay
dnechay merged commit 77db00a into develop Aug 7, 2025
16 checks passed
@github-project-automation github-project-automation Bot moved this from In Review to Done in Core-tech - 2025 Aug 7, 2025
@dnechay dnechay mentioned this pull request Aug 7, 2025
15 tasks
@dnechay
dnechay deleted the dnechay/fs-06082025 branch August 20, 2025 10:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants