Skip to content

Conversation

@sbogolii-sift
Copy link
Contributor

@sbogolii-sift sbogolii-sift commented Nov 7, 2022

These changes were already approved in

#85.

Created this PR to fix build setup on circleci it was not working in prev PR

Sreejith Kalapurakkal and others added 5 commits October 22, 2022 17:52
…Exalture/sift-java into merchant_managemet_api

# Conflicts:
#	src/main/java/com/siftscience/CreateMerchantRequest.java
#	src/main/java/com/siftscience/SiftMerchantRequest.java
#	src/main/java/com/siftscience/UpdateMerchantRequest.java

// Verify the request.
RecordedRequest request1 = server.takeRequest();
Assert.assertEquals("POST", request1.getMethod());
Copy link
Contributor

@sreejithKExalture sreejithKExalture Nov 8, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sbogolii-sift Could you please update "POST" to "PUT" here, as update call will be a PUT. This should resolve the PR build failure. Somehow I am not able to push to this branch.

@viaskal-sift viaskal-sift changed the title added support for merchant management api [MSUE-123] - Added support for merchant management api Nov 8, 2022
@sbogolii-sift sbogolii-sift merged commit d9a43f4 into master Nov 9, 2022
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.

3 participants