Skip to content

Add logs and try-catch for fetching exchange and dispenser details.#1004

Merged
mariacarmina merged 7 commits intofix/index_issuesfrom
bug/fre-processor
Jul 29, 2025
Merged

Add logs and try-catch for fetching exchange and dispenser details.#1004
mariacarmina merged 7 commits intofix/index_issuesfrom
bug/fre-processor

Conversation

@mariacarmina
Copy link
Contributor

@mariacarmina mariacarmina commented Jul 23, 2025

Fixes #1003 .

Changes proposed in this PR:

  • added checks for valid FRE and valid Dispenser contract addresses
  • return existing DDOs for invalid dispenser contract dispenser
  • added explicit logs
Screenshot 2025-07-25 at 17 43 20

@mariacarmina mariacarmina self-assigned this Jul 23, 2025
@mariacarmina mariacarmina marked this pull request as draft July 23, 2025 12:57
@mariacarmina mariacarmina changed the title Add logs and try-catch for fetching exchange details. Add logs and try-catch for fetching exchange and dispenser details. Jul 25, 2025
@mariacarmina mariacarmina marked this pull request as ready for review July 25, 2025 14:47
@alexcos20 alexcos20 changed the base branch from main to fix/index_issues July 29, 2025 05:02
Copy link
Member

@alexcos20 alexcos20 left a comment

Choose a reason for hiding this comment

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

lgtm

@mariacarmina mariacarmina merged commit 4089bb7 into fix/index_issues Jul 29, 2025
13 checks passed
@mariacarmina mariacarmina deleted the bug/fre-processor branch July 29, 2025 08:19
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.

Fix indexing FRE events

2 participants