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

Release to production #10

Merged
merged 65 commits into from
Feb 29, 2020
Merged

Release to production #10

merged 65 commits into from
Feb 29, 2020

Conversation

ghost
Copy link

@ghost ghost commented Feb 29, 2020

No description provided.

jonasanker and others added 30 commits October 12, 2019 12:51
- Removed unnecessary using statements
- Fixed formatting
… ValidateTransaction to accomodate for the new mopbilepay flow
…nged SendEmail in EmailService to private and removed it rfrom interface.

Refactored the remaining methods in EmailService to use RetrieveTemplate
… invoces.

Added a html template for an invoice.

Minor changes in EmailService in regard to invoices
Updated invoice.html so it is now displayed in the intended manner on outlook (desktop)
CamelCase is a dotnet convention
Code Cleanup performed. Various formatting changes.
… ValidateTransaction to accomodate for the new mopbilepay flow
MobilePayService has been rewritten to support new MobilePay flow with Capture and Cancel. MobilePayApiClient introduced which handles the sending and hashing of request.
PurchaseService handles new MobilePay flow where the service is responsible for captuing the amount at MobilePay.
MobilePayService depend on the interface for easier testing
Programme is not set in the app, so new users will have null values.
Currently not implemented
Setup WebApplicationFactory which acts as a mock server
AppSettings are parsed to specialized settings objects which
@jonasanker jonasanker merged commit 3c42a24 into production Feb 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants