Compare commits

...

17 Commits

Author SHA1 Message Date
Nicholas Tindle
b58d7ba8b1 Merge branch 'dev' into seer/fix-postmark-error-handling 2025-10-22 13:18:10 -05:00
Nicholas Tindle
dbadc78240 fix: disable queuing and processing in prod 2025-10-12 21:45:33 -05:00
Nicholas Tindle
6931f40dad fix: bulk set retry to false for notifications 2025-10-12 20:55:18 -05:00
Nicholas Tindle
d4cfb28b18 fix: make it not contentious 2025-10-12 20:52:08 -05:00
Nicholas Tindle
3f430544b6 fix: seperate out the tests for speciifc purposes and test that purpose 2025-10-08 11:56:15 -05:00
Nicholas Tindle
aeb2081df1 feat(backend/notifications): implement user notification management for inactive users 2025-10-07 18:15:08 -05:00
Nicholas Tindle
560441ccfd Merge branch 'dev' into seer/fix-postmark-error-handling 2025-10-07 17:56:23 -05:00
Nicholas Tindle
c0458a10bb Reformat test_notifications.py for improved readability
Refactored line breaks and indentation in test_notifications.py to enhance readability and maintain PEP8 compliance, especially in patch statements and long function calls. No functional changes were made.
2025-10-07 17:54:35 -05:00
Nicholas Tindle
ac10ca43a8 Remove oversized notifications from batch permanently
Oversized notifications that exceed the email size limit are now removed from the batch and will not be retried. This prevents repeated failures and improves batch processing reliability.
2025-10-07 17:52:51 -05:00
Nicholas Tindle
7c03f0556f Merge branch 'seer/fix-postmark-error-handling' of https://github.com/Significant-Gravitas/AutoGPT into seer/fix-postmark-error-handling 2025-10-07 17:38:35 -05:00
Nicholas Tindle
f4576f95d3 fix(backend/notifications): handle processing notifiactions one at a time so we dont resend everything in a batch 2025-10-07 17:38:16 -05:00
claude[bot]
f93128b312 refactor: Update tests to call actual _process_batch implementation
- Refactored all tests to call actual _process_batch method with mocked dependencies
- Fixed AsyncMock import and usage for async functions
- Removed duplicate error handling logic from tests
- Tests now verify real implementation behavior
- Added proper mocking for database operations and internal methods

Co-authored-by: Nicholas Tindle <ntindle@users.noreply.github.com>
2025-10-07 21:39:56 +00:00
Nicholas Tindle
78af66cafb Refactor notification error handling tests
Improves and expands tests for notification error handling by using more realistic data models, patching the logger, and simulating various error scenarios. The tests now verify logging behavior and error handling for inactive recipients, malformed data, oversized notifications, generic errors, mixed error batches, and batch continuation after failures.
2025-10-07 14:42:11 -05:00
Nicholas Tindle
445618a329 Merge branch 'dev' into seer/fix-postmark-error-handling 2025-10-07 10:50:12 -05:00
claude[bot]
c0a5eb10bc feat(notifications): Deactivate email verification on 406 inactive recipient errors
- Add automatic email deactivation when Postmark returns 406 (inactive recipient)
- Mirrors the bounce handler behavior for consistency
- Prevents repeated sending attempts to inactive emails
- Add comprehensive test coverage for the deactivation logic

Co-authored-by: Nicholas Tindle <ntindle@users.noreply.github.com>
2025-10-02 20:54:04 +00:00
claude[bot]
c3938fd320 fix: Improve error handling and add comprehensive tests for notification failures
- Fixed unused variable issue that was failing linting
- Improved error detection with more robust checking instead of fragile string matching
- Removed invalid PostmarkException imports (the library doesn't export these)
- Added comprehensive unit tests for all error scenarios:
  - 406 inactive recipient errors
  - 422 malformed data errors
  - Oversized notification errors
  - Generic API errors
  - Mixed error scenarios
  - Batch processing continuation after failures
- Updated error handling to catch generic exceptions and check error messages
- Tests verify proper logging and error handling behavior

Co-authored-by: Nicholas Tindle <ntindle@users.noreply.github.com>
2025-10-02 20:36:54 +00:00
seer-by-sentry[bot]
ec36af2c45 fix(backend): Improve Postmark error handling and logging for notification delivery 2025-10-02 16:43:01 +00:00

Diff Content Not Available