Skip to content

fix(frontend): stabilize cash-flow forecast loading - #678

Open
woahwhattheheck wants to merge 2 commits into
Protocol-Guild:mainfrom
woahwhattheheck:sol56/payd542-cashflow-auth-20261004
Open

woahwhattheheck wants to merge 2 commits into
Protocol-Guild:mainfrom
woahwhattheheck:sol56/payd542-cashflow-auth-20261004

Conversation

@woahwhattheheck

Copy link
Copy Markdown

Closes #542

Summary

  • Avoid mount-time empty-parameter fetch/toast behavior by leaving forecast loading to the explicit Refresh path.
  • Read the canonical payd_auth_token key and omit the Authorization header when no token is present.
  • Keep valid stored tokens in Bearer authorization for cash-flow API calls.
  • Stabilize forecast callback dependencies on primitive account/issuer/days values plus notification functions rather than a mutable params object.

Validation

The exact two-file carrier was read back against sponsor base with zero accessToken reads in cashFlowForecastApi.ts, no parameter-driven useEffect in CashFlowForecast.tsx, and the primitive callback dependency list present. The source environment could not materialize the repository for a local build, so no build or CI pass is claimed.

Source: woahwhattheheck/PayD:sol56/payd542-cashflow-auth-20261004 at ccb04f99bf1daa10b81892ffae00231f2aeea5a9.

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.

Fix CashFlowForecast infinite re-render risk and wrong auth token key

1 participant