Skip to content

Conversation

@nesitor
Copy link
Member

@nesitor nesitor commented Jun 19, 2025

Fix: Solve issue with not rejecting messages that it's content doesn't exist on IPFS

Self proofreading checklist

  • Is my code clear enough and well documented
  • Are my files well typed
  • New translations have been added or updated if new strings have been introduced in the frontend
  • Database migrations file are included
  • Are there enough tests
  • Documentation has been included (for new feature)

Changes

Added a new exception to specify IPFS content not found and reject the message after 10 retries as usual.

@nesitor nesitor requested a review from amalcaraz June 19, 2025 10:29
@nesitor nesitor self-assigned this Jun 19, 2025
amalcaraz
amalcaraz previously approved these changes Jun 19, 2025
Copy link
Collaborator

@amalcaraz amalcaraz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Collaborator

@amalcaraz amalcaraz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@nesitor nesitor merged commit 3cf1938 into main Jun 19, 2025
5 checks passed
@nesitor nesitor deleted the andres-fix-solve_ipfs_content_not_found branch June 19, 2025 11:15
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.

3 participants