Skip to content

fix(rpc): more fixes in filecoin chain notify API - #7502

Draft
akaladarshi wants to merge 3 commits into
mainfrom
akaladarshi/fix-more-chain-notify-issues
Draft

fix(rpc): more fixes in filecoin chain notify API#7502
akaladarshi wants to merge 3 commits into
mainfrom
akaladarshi/fix-more-chain-notify-issues

Conversation

@akaladarshi

Copy link
Copy Markdown
Contributor

Summary of changes

Changes introduced in this pull request:

  • Removing the closed subscription from the registry

Reference issue to close (if applicable)

Partially Closes #5795

Other information and links

Change checklist

  • I have performed a self-review of my own code,
  • I have made corresponding changes to the documentation. All new code adheres to the team's documentation standards,
  • I have added tests that prove my fix is effective or that my feature works (if possible),
  • I have made sure the CHANGELOG is up-to-date. All user-facing changes should be reflected in this document.

Outside contributions

  • This pull request is based on an issue that a maintainer has accepted (see Before Opening a Pull Request).
  • I have read and agree to the CONTRIBUTING document.
  • I have read and agree to the AI Policy document. I understand that failure to comply with the guidelines will lead to rejection of the pull request.

@akaladarshi
akaladarshi requested a review from a team as a code owner August 18, 2026 13:52
@akaladarshi
akaladarshi requested review from EclesioMeloJunior and LesnyRumcajs and removed request for a team August 18, 2026 13:52
@akaladarshi
akaladarshi marked this pull request as draft August 18, 2026 13:52
@coderabbitai

coderabbitai Bot commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

鈿欙笍 Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: bc9404f9-91fe-4325-ab9a-3dabfa6a38f9

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 馃攳 Trigger review

Comment @coderabbitai help to get the list of available commands.

@akaladarshi
akaladarshi force-pushed the akaladarshi/fix-chain-notify-and-tests branch from 1d3d05b to 75201ec Compare August 18, 2026 16:29
Base automatically changed from akaladarshi/fix-chain-notify-and-tests to main August 20, 2026 17:05
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.

1 participant