Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

🐛 improved gocardless error handling #282

Merged
merged 2 commits into from
Dec 1, 2023
Merged

🐛 improved gocardless error handling #282

merged 2 commits into from
Dec 1, 2023

Conversation

MatissJanis
Copy link
Member 

Error /gocardless/transactions TypeError: Cannot read properties of undefined (reading 'data')
    at file:///app/src/app-gocardless/app-gocardless.js:208:36
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)

@trafico-bot trafico-bot bot added ✅ Approved Pull Request has been approved and can be merged and removed 🔍 Ready for Review labels Dec 1, 2023
@MatissJanis MatissJanis merged commit dc47c6d into master Dec 1, 2023
8 checks passed
@MatissJanis MatissJanis deleted the matiss/err-handling branch December 1, 2023 17:49
@trafico-bot trafico-bot bot added ✨ Merged Pull Request has been merged successfully and removed ✅ Approved Pull Request has been approved and can be merged labels Dec 1, 2023
MMichotte pushed a commit to MMichotte/actual-server that referenced this pull request Sep 9, 2024
@MatissJanis @MMichotte
🐛 improved gocardless error handling (actualbudget#282)
d1ed483
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@joel-jeremy joel-jeremy joel-jeremy approved these changes

Assignees
No one assigned
Labels
✨ Merged Pull Request has been merged successfully
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@MatissJanis @joel-jeremy