Release 7.1.0 #363
Merged
Release 7.1.0 #363
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* PW-2080: Add webhooks event codes for additional configuration * Update src/test/resources/mocks/notification/offer-closed.json
* PW-2378: added riskdata to checkout payments request * PW-2378: fix comment * PW-2378: remove systemout
* PW-1999: added support for StoredValue API * PW-1999: voidTransaction * PW-1999: checkstyle changes only * PW-1999: API_VERSION * PW-1999: single class imports * PW-1999: changed paymentMethod type in requests, added securitycode to DefaultPaymentMethodDetails, added tests for Issue and ChangeStatus * PW-1999: added remaining unit tests * PW-1999: fix comment
* Remove redundant local variable * Remove unnecessary conversion * lowercase packagename Util to util * Replace null check with Objects.isNull * - Create TextConstants class - Replace "null" String value with TextConstants.NULL * - Create TextConstants class - Replace "null" String value with TextConstants.NULL * Make use of Util.toIndentedString method - replace duplicate code - replace \n with constant * static import Util.toIndentedString * static import TextConstatnts.LINE_BREAK * Assign new object to local variable * Assign new object to local variable * Revert Objects.isNull check * Revert Objects.isNull check * Enable the InnerAssignment checkstyle again Co-authored-by: andrei-dediu <59608940+andrei-dediu@users.noreply.github.com>
Aleffio
requested review from
AlexandrosMor,
andrei-dediu,
martinsrenato,
rikterbeek and
rkewlani
as code owners
|
This pull request fixes 2 alerts when merging fe79acb into 95e480e - view on LGTM.com fixed alerts:
|
andrei-dediu
previously approved these changes
Jun 24, 2020
martinsrenato
previously approved these changes
Jun 24, 2020
revert null, line_break and line_break_with_whitespace constants
|
This pull request fixes 2 alerts when merging 7c2deb3 into 95e480e - view on LGTM.com fixed alerts:
|
|
This pull request fixes 2 alerts when merging a5b2d44 into 95e480e - view on LGTM.com fixed alerts:
|
rkewlani
approved these changes
Jun 26, 2020
andrei-dediu
approved these changes
Jun 26, 2020
|
this unrelease is currently blocking a launch due to the "riskData" parameter missing issue, any speed here is greatly appreciated |
|
Hi @jonathanselander , apologies for the delay, Regards, |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
#358 Add webhooks event codes for additional configuration
#359 added support for StoredValue API
#360 added riskdata to checkout payments request
#361 add brand and networkPaymentReference fields to support Network Tokenization
#362 add idempotency key for instant payouts
Tested scenarios
Fixed issue: