Fix submitProposal (#4535)

open-chat-labs · Oct 9, 2023 · 64be779 · 64be779
1 parent 6156377
commit 64be779
Show file tree

Hide file tree

Showing 6 changed files with 7 additions and 10 deletions.
diff --git a/frontend/app/src/components/home/MakeProposalModal.svelte b/frontend/app/src/components/home/MakeProposalModal.svelte
@@ -106,10 +106,10 @@
     function onClickPrimary() {
         if (step === 0) {
             balanceWithRefresh.refresh();
+        } else if (canSubmit) {
+            onSubmit();
         } else if (step === 1) {
             step = 2;
-        } else if (step === 2) {
-            onSubmit();
         }
     }
 

diff --git a/frontend/openchat-agent/src/services/openchatAgent.ts b/frontend/openchat-agent/src/services/openchatAgent.ts
@@ -2548,7 +2548,7 @@ export class OpenChatAgent extends EventTarget {
         }
     }
 
-    submitProposal(governanceCanisterId: Principal, proposal: CandidateProposal): Promise<SubmitProposalResponse> {
+    submitProposal(governanceCanisterId: string, proposal: CandidateProposal): Promise<SubmitProposalResponse> {
         return this.userClient.submitProposal(governanceCanisterId, proposal);
     }    
 }
diff --git a/frontend/openchat-agent/src/services/user/mappers.ts b/frontend/openchat-agent/src/services/user/mappers.ts
@@ -1079,7 +1079,6 @@ function proposalAction(action: CandidateProposalAction): ProposalToSubmitAction
                 treasury: action.treasury === "ICP" ? { ICP: null } : { SNS: null }
             }};
     }
-    throw new Error("Unexpected proposal action");
 }
 
 export function submitProposalResponse(candid: ApiSubmitProposalResponse): SubmitProposalResponse {

diff --git a/frontend/openchat-agent/src/services/user/user.client.ts b/frontend/openchat-agent/src/services/user/user.client.ts
@@ -1121,10 +1121,10 @@ export class UserClient extends CandidService {
         );
     }
 
-    submitProposal(governanceCanisterId: Principal, proposal: CandidateProposal): Promise<SubmitProposalResponse> {
+    submitProposal(governanceCanisterId: string, proposal: CandidateProposal): Promise<SubmitProposalResponse> {
         return this.handleResponse(
             this.userService.submit_proposal({
-                governance_canister_id : governanceCanisterId,
+                governance_canister_id : Principal.fromText(governanceCanisterId),
                 proposal : proposalToSubmit(proposal),
             }),
             submitProposalResponse,

diff --git a/frontend/openchat-client/src/openchat.ts b/frontend/openchat-client/src/openchat.ts
@@ -1,5 +1,4 @@
 /* eslint-disable no-case-declarations */
-import { Principal } from "@dfinity/principal";
 import type { Identity } from "@dfinity/agent";
 import { AuthClient } from "@dfinity/auth-client";
 import { get, writable } from "svelte/store";
@@ -4998,7 +4997,7 @@ export class OpenChat extends OpenChatAgentWorker {
     submitProposal(governanceCanisterId: string, proposal: CandidateProposal): Promise<boolean> {
         return this.sendRequest({
             kind: "submitProposal",
-            governanceCanisterId: Principal.fromText(governanceCanisterId),
+            governanceCanisterId,
             proposal,
         })
             .then((resp) => {

diff --git a/frontend/openchat-shared/src/domain/worker.ts b/frontend/openchat-shared/src/domain/worker.ts
@@ -139,7 +139,6 @@ import type {
 import type { ChatPermissions } from "./permission";
 import type { RegistryValue } from "./registry";
 import type { StakeNeuronForSubmittingProposalsResponse } from "./proposalsBot";
-import type { Principal } from "@dfinity/principal";
 import type { CandidateProposal } from "./proposals";
 /**
  * Worker request types
@@ -1326,7 +1325,7 @@ type FollowThread = {
 };
 
 type SubmitProposal = {
-    governanceCanisterId: Principal;
+    governanceCanisterId: string;
     proposal: CandidateProposal;
     kind: "submitProposal";
 };
-Original file line number
+Diff line change
@@ Expand Up @@
                     treasury: action.treasury === "ICP" ? { ICP: null } : { SNS: null }
                 }};
         }
-        throw new Error("Unexpected proposal action");
     }
     export function submitProposalResponse(candid: ApiSubmitProposalResponse): SubmitProposalResponse {
@@ Expand Down @@