Skip to content
This repository was archived by the owner on Apr 6, 2022. It is now read-only.

payPledge via webhook for postfinance #26

Merged
merged 4 commits into from
Apr 16, 2018
Merged

payPledge via webhook for postfinance #26

merged 4 commits into from
Apr 16, 2018

Conversation

patte
Copy link
Contributor

@patte patte commented Apr 13, 2018 •
edited
Loading

disclaimer: no tests because we don't have any tests for POSTFINANCE yet, and setting all of this up would blast the scope of this bugfix.

@patte patte requested a review from tpreusse April 13, 2018 17:23
@patte patte temporarily deployed to republik-api-staging April 13, 2018 17:26 Inactive
@coveralls
Copy link

coveralls commented Apr 13, 2018 •
edited
Loading

Pull Request Test Coverage Report for Build 138

  • 0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 85.965%
Totals Coverage Status
Change from base Build 126: 0.0%
Covered Lines: 44
Relevant Lines: 47
💛 - Coveralls

@patte patte temporarily deployed to republik-api-staging April 16, 2018 15:26 Inactive
@patte patte merged commit 06bcd1a into master Apr 16, 2018
@patte patte deleted the pf-webhook branch April 16, 2018 16:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
1 more reviewer

@tpreusse tpreusse tpreusse approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@patte @coveralls @tpreusse