Skip to content

Commit

Permalink
fix: merge votes button not working (#2466)
Browse files Browse the repository at this point in the history
* fix: removed non-existing wallet.rs function

* chore: remove unused function
  • Loading branch information
Tuditi authored Mar 4, 2022
1 parent ace7916 commit 7f7ad99
Show file tree
Hide file tree
Showing 5 changed files with 2 additions and 76 deletions.
8 changes: 0 additions & 8 deletions packages/backend/bindings/node/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,6 @@ import {
getParticipationEvents as _getParticipationEvents,
getParticipationOverview as _getParticipationOverview,
participate as _participate,
participateWithRemainingFunds as _participateWithRemainingFunds,
stopParticipating as _stopParticipating,
} from '../../../shared/lib/participation/bridge'
import { Participation } from '../../../shared/lib/participation/types'
Expand Down Expand Up @@ -369,13 +368,6 @@ export const api = {
stopParticipating: function (accountId: string, eventIds: string[]): (__ids: CommunicationIds) => Promise<string> {
return (__ids: CommunicationIds) => _stopParticipating(sendMessage, __ids, accountId, eventIds)
},
participateWithRemainingFunds: function (
accountId: string,
participations: Participation[]
): (__ids: CommunicationIds) => Promise<string> {
return (__ids: CommunicationIds) =>
_participateWithRemainingFunds(sendMessage, __ids, accountId, participations)
},

// Event emitters
onError: function (): (__ids: CommunicationIds) => Promise<string> {
Expand Down
10 changes: 2 additions & 8 deletions packages/shared/components/popups/GovernanceCastVote.svelte
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
import { closePopup } from 'shared/lib/popup'
import { isSoftwareProfile } from 'shared/lib/profile'
import { selectedAccount, api, isSyncing } from 'shared/lib/wallet'
import { participate, participateWithRemainingFunds, stopParticipating } from 'shared/lib/participation/api'
import { participate, stopParticipating } from 'shared/lib/participation/api'
import { showAppNotification } from 'shared/lib/notifications'
import { openPopup } from 'shared/lib/popup'
import {
Expand Down Expand Up @@ -82,14 +82,8 @@
const vote = async (): Promise<void> => {
switch (votingAction) {
case VotingAction.Cast:
await participate(
$selectedAccount?.id,
[{ eventId, answers: [nextVote?.value] }],
ParticipationAction.Vote
)
break
case VotingAction.Merge:
await participateWithRemainingFunds(
await participate(
$selectedAccount?.id,
[{ eventId, answers: [nextVote?.value] }],
ParticipationAction.Vote
Expand Down
23 changes: 0 additions & 23 deletions packages/shared/lib/participation/api.ts
Original file line number Diff line number Diff line change
Expand Up @@ -106,26 +106,3 @@ export function stopParticipating(
})
})
}

export function participateWithRemainingFunds(
accountId: string,
participations: Participation[],
action: ParticipationAction
): Promise<string[]> {
return new Promise((resolve, reject) => {
api.participateWithRemainingFunds(accountId, participations, {
onSuccess(response: Event<ParticipateResponsePayload>) {
response.payload.forEach((message) => saveNewMessage(accountId, message))

addNewPendingParticipation(response.payload, accountId, action)

resolve(response.payload.map((message) => message.id))
},
onError(error) {
console.error(error)

reject(error)
},
})
})
}
29 changes: 0 additions & 29 deletions packages/shared/lib/participation/bridge.ts
Original file line number Diff line number Diff line change
Expand Up @@ -96,32 +96,3 @@ export function stopParticipating(
},
})
}

/**
* Participate in event(s) with additional funds
*
* @method participateWithRemainingFunds
*
* @param {Bridge} bridge
* @param {CommunicationIds} __ids
* @param {string} accountId
* @param {Participation[]} participations
*
* @returns {Promise<string>}
*/
export function participateWithRemainingFunds(
bridge: Bridge,
__ids: CommunicationIds,
accountId: string,
participations: Participation[]
): Promise<string> {
return bridge({
actorId: __ids.actorId,
id: __ids.messageId,
cmd: 'ParticipateWithRemainingFunds',
payload: {
account_identifier: accountId,
participations,
},
})
}
8 changes: 0 additions & 8 deletions packages/shared/lib/typings/walletApi.ts
Original file line number Diff line number Diff line change
Expand Up @@ -291,12 +291,4 @@ export interface IWalletApi {
onError: (err: ErrorEventPayload) => void
}
)
participateWithRemainingFunds(
accountId: string,
participations: Participation[],
callbacks: {
onSuccess: (response: Event<ParticipateResponsePayload>) => void
onError: (err: ErrorEventPayload) => void
}
)
}

0 comments on commit 7f7ad99

Please sign in to comment.