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.

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

Fix bug in transaction listing for admins #187

Merged
merged 1 commit into from Sep 10, 2019

Conversation

@mraerino
Copy link
Member

commented Sep 10, 2019

Fixes #181

- Summary

Bugfix related to giving db driver too many parameters

- Test plan

Hard to test a regression because it does not show up in sqlite...

- Description for the changelog

Fix error on /payments endpoint.

@rybit
rybit approved these changes Sep 10, 2019

@mraerino mraerino force-pushed the fix/transaction-list-error branch from 49d93a9 to 0d4442b Sep 10, 2019

@mraerino mraerino merged commit ac25dbd into master Sep 10, 2019

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@mraerino mraerino deleted the fix/transaction-list-error branch Sep 10, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.