chore: bump @ecency/sdk to 2.3.83 and drop the wallet history initialData guard - #3493
Conversation
…Data guard sdk 2.3.83 ships a spec pinning that the notifications and HIVE/HBD/HP asset transaction options carry no initialData seed, which is the exact condition the local guard documented for its own removal.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThe PR updates ChangesWallet history contract
Estimated code review effort: 1 (Trivial) | ~5 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Raises the @ecency/sdk floor to ^2.3.83 (published from ecency/vision-web#1427) and removes the
initialData: undefinedoverride on the wallet history chain query. The guard's own comment gated its removal on an SDK spec pinning the absence of the empty seed; 2.3.83 ships that spec for both the notifications options and the HIVE/HBD/HP asset transaction options, so the contract is enforced upstream now.Verified: typecheck clean against the empty baseline, lint 0 errors, 821 jest tests pass.
Closes #3492
Summary by CodeRabbit