Skip to content
This repository has been archived by the owner on Dec 22, 2021. It is now read-only.

[PAN-2893] [PAN-2892] Rename privacyGroupId API to createPrivacyGroupId #251

Merged
merged 3 commits into from
Jul 8, 2019

Conversation

Puneetha17
Copy link
Contributor

@Puneetha17 Puneetha17 commented Jul 8, 2019

Also, fixes the issue PAN-2892

@Puneetha17 Puneetha17 changed the title Rename privacyGroupId API to createPrivacyGroupId [PAN-2892] Rename privacyGroupId API to createPrivacyGroupId Jul 8, 2019
@Puneetha17 Puneetha17 changed the title [PAN-2892] Rename privacyGroupId API to createPrivacyGroupId [PAN-2893] [PAN-2892] Rename privacyGroupId API to createPrivacyGroupId Jul 8, 2019
Copy link
Contributor

@iikirilov iikirilov left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -141,7 +141,7 @@ ReceiveResponse receivePrivacy(String digest, String publicKey) {
public Optional<PrivacyGroup> createPrivacyGroup(String[] addresses, String from, String name, String description) {
PrivacyGroupRequest createGroupRequest = new PrivacyGroupRequest(addresses, from, name, description);
CompletableFuture<PrivacyGroup> keyFuture = new CompletableFuture<>();
httpClient.post(clientPort, "localhost", "/privacyGroupId").handler(resp -> {
httpClient.post(clientPort, "localhost", "/createPrivacyGroupId").handler(resp -> {
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
httpClient.post(clientPort, "localhost", "/createPrivacyGroupId").handler(resp -> {
httpClient.post(clientPort, "localhost", "/createPrivacyGroup").handler(resp -> {

networkNodes,
enclave,
vertx,
config));

clientRouter.post("/deletePrivacyGroupId").consumes(JSON.httpHeaderValue).produces(JSON.httpHeaderValue).handler(
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
clientRouter.post("/deletePrivacyGroupId").consumes(JSON.httpHeaderValue).produces(JSON.httpHeaderValue).handler(
clientRouter.post("/deletePrivacyGroup").consumes(JSON.httpHeaderValue).produces(JSON.httpHeaderValue).handler(

Copy link
Contributor

@ekellstrand ekellstrand left a comment

Choose a reason for hiding this comment

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

LGTM

@ekellstrand ekellstrand merged commit 27158e2 into Consensys:master Jul 8, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants