Skip to content

Fix multiple issues (#449, #475, #476, #477)#1

Open
software321dev wants to merge 1 commit into
mainfrom
fix-multiple-issues
Open

Fix multiple issues (#449, #475, #476, #477)#1
software321dev wants to merge 1 commit into
mainfrom
fix-multiple-issues

Conversation

@software321dev

Copy link
Copy Markdown
Owner

This PR addresses multiple outstanding issues across the backend and frontend to improve performance, caching, security, and user experience.

Changes

  • Backend (Payments): Refactored fetchPaymentEvents to compute a startLedger approximation based on the days parameter. This prevents the Soroban RPC from scanning from the genesis ledger on every request.
  • Backend (Caching): Updated the cacheFor middleware to properly set the Cache-Control: public, max-age=... response header, ensuring clients and proxies correctly cache the response.
  • Backend (Security): Added the requireAdminKey middleware guard to the GET /api/meters/export route to prevent unauthorized dumping of all meter data.
  • Frontend (UI/UX): Replaced the empty data state in the Provider Dashboard meter table with SkeletonCard placeholders during data loading. Also added proper error state handling.

Fixes

Closes Dev-AdeTutu#449
Closes Dev-AdeTutu#475
Closes Dev-AdeTutu#476
Closes Dev-AdeTutu#477

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment