fix(rpc): make ChainGetTipset v2 work with lotus-gateway#6553
fix(rpc): make ChainGetTipset v2 work with lotus-gateway#6553hanabi1224 wants to merge 5 commits intomainfrom
Conversation
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (1)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including You can disable this status message by setting the
WalkthroughAdds serde defaults for TipsetSelector fields, normalizes lotus-gateway-wrapped RPC error messages in API comparison tests, updates test endpoints to WebSocket, and adds related tests and changelog entry. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@CHANGELOG.md`:
- Line 66: Update the changelog entry that currently references issue `#6553` so
it points to issue `#6552` instead: replace the link and number in the line "-
[`#6553`](https://github.com/ChainSafe/forest/issues/6553):
`Filecoin.ChainGetTipset` now works with `lotus-gateway`." to use "#6552" and
the URL "https://github.com/ChainSafe/forest/issues/6552" so the entry reads "-
[`#6552`](https://github.com/ChainSafe/forest/issues/6552):
`Filecoin.ChainGetTipset` now works with `lotus-gateway`."
|
no green checkmark, no review |
Summary of changes
Changes introduced in this pull request:
Reference issue to close (if applicable)
Closes #6552
Other information and links
Change checklist
Outside contributions
Summary by CodeRabbit
New Features
Improvements
Documentation
Tests