Skip to content

Conversation

@bernd2022
Copy link

  • fix: Track all dEURO mints including CoW Protocol swaps
  • Add getRecentMints() method to query all mints from Ponder
  • Extend Telegram service to send notifications for all significant mints
  • Use correct Ponder GraphQL syntax without where clause filtering
  • Add chain-specific transaction explorer URLs

This ensures all dEURO mints trigger social media notifications, not just those with MintingUpdateV2 events.

  • chore: Remove unnecessary package-lock.json and restore yarn.lock

  • fix: Prevent duplicate notifications and improve error handling

  • Add duplicate detection to avoid sending same mint twice
  • Track MintingUpdateV2 txHashes to filter them from general mints
  • Add try/catch for error handling when fetching mints
  • Increase query limit from 100 to 500 to handle more mints
  • Use earlier timestamp between generalMints and mintingUpdates to avoid gaps
  • fix: Critical bugs in mint notification system
  • Fix initial timestamp bug: Use current time instead of future time for mint tracking
  • Improve duplicate detection: Check txHash existence before processing
  • Simplify timestamp logic: Remove problematic Math.min() that caused missed mints
  • Add proper txHash validation before using in URLs
  • Clean up filter logic for better readability
  • fix: Prevent notification spam on restart and limit batch size
  • Initialize mint tracking from 1 hour ago instead of current time
  • Prevents duplicate notifications after API restart/deployment
  • Add MAX_NOTIFICATIONS limit (10) per update cycle to prevent spam
  • Log warning when notification limit is reached
  • Balances between catching recent mints and avoiding notification floods
  • fix: Simplify notification message - remove unnecessary URL construction
  • Remove chain-specific URL logic (dEURO only exists on Polygon)
  • Display txHash directly in message instead of constructing URLs
  • Simpler and cleaner notification format
  • fix: Remove unnecessary complexity from mint tracking
  • Reduce query limit from 500 to 50 (sufficient for regular updates)
  • Remove unused fields from query (id, blockheight)
  • Start tracking from current time instead of 1 hour ago
  • Simplifies implementation without affecting functionality
  • fix: Radically simplify mint tracking implementation
  • Remove all duplicate detection logic (accept occasional duplicates)
  • Remove separate generalMints timestamp tracking
  • Remove MAX_NOTIFICATIONS limit and logging
  • Remove try-catch and error handling
  • Remove txHash from types and queries
  • Simplify getRecentMints to minimal implementation
  • Reuse existing mintingUpdates timestamp for all mints

From 106 lines to ~25 lines of actual new code

  • feat: Adapt to the existing implementation

* fix: Track all dEURO mints including CoW Protocol swaps

- Add getRecentMints() method to query all mints from Ponder
- Extend Telegram service to send notifications for all significant mints
- Use correct Ponder GraphQL syntax without where clause filtering
- Add chain-specific transaction explorer URLs

This ensures all dEURO mints trigger social media notifications, not just those with MintingUpdateV2 events.

* chore: Remove unnecessary package-lock.json and restore yarn.lock

* fix: Prevent duplicate notifications and improve error handling

- Add duplicate detection to avoid sending same mint twice
- Track MintingUpdateV2 txHashes to filter them from general mints
- Add try/catch for error handling when fetching mints
- Increase query limit from 100 to 500 to handle more mints
- Use earlier timestamp between generalMints and mintingUpdates to avoid gaps

* fix: Critical bugs in mint notification system

- Fix initial timestamp bug: Use current time instead of future time for mint tracking
- Improve duplicate detection: Check txHash existence before processing
- Simplify timestamp logic: Remove problematic Math.min() that caused missed mints
- Add proper txHash validation before using in URLs
- Clean up filter logic for better readability

* fix: Prevent notification spam on restart and limit batch size

- Initialize mint tracking from 1 hour ago instead of current time
- Prevents duplicate notifications after API restart/deployment
- Add MAX_NOTIFICATIONS limit (10) per update cycle to prevent spam
- Log warning when notification limit is reached
- Balances between catching recent mints and avoiding notification floods

* fix: Simplify notification message - remove unnecessary URL construction

- Remove chain-specific URL logic (dEURO only exists on Polygon)
- Display txHash directly in message instead of constructing URLs
- Simpler and cleaner notification format

* fix: Remove unnecessary complexity from mint tracking

- Reduce query limit from 500 to 50 (sufficient for regular updates)
- Remove unused fields from query (id, blockheight)
- Start tracking from current time instead of 1 hour ago
- Simplifies implementation without affecting functionality

* fix: Radically simplify mint tracking implementation

- Remove all duplicate detection logic (accept occasional duplicates)
- Remove separate generalMints timestamp tracking
- Remove MAX_NOTIFICATIONS limit and logging
- Remove try-catch and error handling
- Remove txHash from types and queries
- Simplify getRecentMints to minimal implementation
- Reuse existing mintingUpdates timestamp for all mints

From 106 lines to ~25 lines of actual new code

* feat: Adapt to the existing implementation

---------

Co-authored-by: bernd2022 <bernd.nospam.2018@t-online.de>
@bernd2022 bernd2022 merged commit 50f00eb into main Aug 21, 2025
1 check passed
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