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

ECO-1341 PayNow for credit cards #17

Merged
merged 53 commits into from
Jul 5, 2018

Conversation

aleksey-kotsuba
Copy link
Contributor

Please confirm

  • No new OS components - or they have been approved by legal department
  • All new or heavily changed classes get an "A" rating (Scrutinizer), except Facades, Factories, QueryContainers and Tests

Documentation

  • Functional documentation provided or in progress?
  • Integration guide for projects provided or not needed?
  • Migration guides for all contained majors provided or not needed?

Release Table

Module Release Type Constraint Updates
Computop major (new)

Release Notes

  • Add new PayNow payment method.

Module Computop

Maxi-Major: New or fully refactored module.

Def of done (by responsible developer):

  • All methods in facade, client and plugin interfaces provide an API-doc.
  • All dependencies are defined, checked and corrected.
  • There are no violations of the architecture rules. All cleanups checked.
  • All new or changed business logic is covered by functional tests (in Zed business layer).
  • Migration guide provided (for existing modules).
  • New modules have module description in README, for existing ones they need to be up to date.
Change log

Improvements

  • Add new PayNow payment method.

Aleksey Kotsuba added 30 commits April 18, 2018 13:43
@alex-galych
Copy link
Contributor

Will be reviewed by Architect in release PR.

@alex-galych alex-galych merged commit 5236b49 into dev Jul 5, 2018
@aleksey-kotsuba aleksey-kotsuba deleted the feature/eco-1341-paynow-integration branch November 19, 2018 18:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants