sui icon indicating copy to clipboard operation
sui copied to clipboard

[indexer alt] - make ClientArgs derive default

Open patrickkuo opened this issue 6 months ago • 1 comments

Description

Make ClientArg derive Default so we can omit optional fields.

Test plan

How did you test the new or updated feature?


Release notes

Check each box that your changes affect. If none of the boxes relate to your changes, release notes aren't required.

For each box you select, include information after the relevant heading that describes the impact of your changes that a user might notice and any actions they must take to implement updates.

  • [ ] Protocol:
  • [ ] Nodes (Validators and Full nodes):
  • [ ] gRPC:
  • [ ] JSON-RPC:
  • [ ] GraphQL:
  • [ ] CLI:
  • [ ] Rust SDK:

patrickkuo avatar Jun 03 '25 10:06 patrickkuo

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

Name Status Preview Comments Updated (UTC)
sui-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 3, 2025 10:25am
2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
multisig-toolkit ⬜️ Ignored (Inspect) Jun 3, 2025 10:25am
sui-kiosk ⬜️ Ignored (Inspect) Jun 3, 2025 10:25am

vercel[bot] avatar Jun 03 '25 10:06 vercel[bot]

This PR is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days.

github-actions[bot] avatar Aug 04 '25 02:08 github-actions[bot]

@tpham-mysten ended up introducing a builder pattern which means we no longer need to go down this route!

amnn avatar Aug 08 '25 19:08 amnn