feat(miden): configure 0.16 testnet dummy faucets - #10
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
React Doctor found 4 new issues in 4 files · 4 warnings · score 84 / 100 (Needs work) · 3 fixed · vs 4 warnings
Reviewed by React Doctor for commit |
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
| @@ -117,6 +118,7 @@ function StatusRow({ | |||
| } | |||
|
|
|||
| export function IntentStatus({ result, error, flowStatus, isPolling }: Props) { | |||
There was a problem hiding this comment.
React Doctor · react-doctor/no-high-complexity-react-function (warning)
IntentStatus has cyclomatic complexity 18, cognitive complexity 17, and maximum nesting depth 2, so its React logic is hard to understand and change. Extract independent branches into components or hooks.
Fix → Extract independent render branches and state logic into focused components or hooks until the control flow is easy to follow.
| const networkConfig = getMidenNetworkConfig(network); | ||
|
|
||
| return ( | ||
| <MidenNetworkContext.Provider value={{ network }}> |
There was a problem hiding this comment.
React Doctor · react-doctor/jsx-no-constructed-context-values (warning)
Every reader of this context redraws on each render because you build its value inline.
Fix → Wrap the context value in useMemo or move it outside the component so consumers do not redraw every render.
This branch was successfully deployed
2 participants
Uses the tested Miden Testnet USDC dummy faucet as the default and recognizes all five configured 6-decimal dummy faucet assets.