From 9de9313ffde5be16261ac70af1f65a1947c0fa86 Mon Sep 17 00:00:00 2001 From: Illya Gerasymchuk Date: Thu, 19 Dec 2024 16:41:29 +0000 Subject: [PATCH 1/2] chore: bump verstion to 0.1.25 --- package-lock.json | 2 +- package.json | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/package-lock.json b/package-lock.json index 0a1af75..28bc8fd 100644 --- a/package-lock.json +++ b/package-lock.json @@ -1,6 +1,6 @@ { "name": "pgt-web-app", - "version": "0.1.24", + "version": "0.1.25", "lockfileVersion": 3, "requires": true, "packages": { diff --git a/package.json b/package.json index 561f194..311dd7a 100644 --- a/package.json +++ b/package.json @@ -1,6 +1,6 @@ { "name": "pgt-web-app", - "version": "0.1.24", + "version": "0.1.25", "private": true, "type": "module", "scripts": { From e168c08443703ffb0da6d48e37e800b0d17e1a4d Mon Sep 17 00:00:00 2001 From: Illya Gerasymchuk Date: Thu, 19 Dec 2024 16:41:46 +0000 Subject: [PATCH 2/2] feat: fixup OCV response (typo in eligible) --- src/services/OCVApiService.ts | 2 +- src/services/ProposalStatusMoveService.ts | 18 +++++++++--------- src/tasks/ocv-vote-counting.ts | 2 +- 3 files changed, 11 insertions(+), 11 deletions(-) diff --git a/src/services/OCVApiService.ts b/src/services/OCVApiService.ts index ca51f65..19976a5 100644 --- a/src/services/OCVApiService.ts +++ b/src/services/OCVApiService.ts @@ -19,7 +19,7 @@ export interface OCVVoteResponse { positive_stake_weight: string; negative_stake_weight: string; vote_status: string; - eligible: boolean; + elegible: boolean; votes: OCVVote[]; } diff --git a/src/services/ProposalStatusMoveService.ts b/src/services/ProposalStatusMoveService.ts index e2874da..ce33536 100644 --- a/src/services/ProposalStatusMoveService.ts +++ b/src/services/ProposalStatusMoveService.ts @@ -61,7 +61,7 @@ interface ProposalWithVotes extends ProposalWithVotesBase { interface MoveResult { newStatus: ProposalStatus; - ocvEligible: boolean; + ocvElegible: boolean; reviewerVotesGiven: number; reviewerVotesRequired: number; } @@ -107,13 +107,13 @@ export class ProposalStatusMoveService { const numReviewerApprovals = this.countValidApprovals(proposal); const ocvData = proposal.OCVConsiderationVote?.voteData as OCVVoteResponse | undefined; - const isEligible = ocvData?.eligible ?? false; + const isElegible = ocvData?.elegible ?? false; const thresholdReviewerApprovals = this.config.considerationPhase.minReviewerApprovals; return { newStatus, - ocvEligible: isEligible, + ocvElegible: isElegible, reviewerVotesGiven: numReviewerApprovals, reviewerVotesRequired: thresholdReviewerApprovals }; @@ -158,24 +158,24 @@ export class ProposalStatusMoveService { const approvalCount = this.countValidApprovals(proposal); const isMinApprovals = approvalCount >= this.config.considerationPhase.minReviewerApprovals; const ocvData = proposal.OCVConsiderationVote?.voteData as OCVVoteResponse | undefined; - const ocvEligible = ocvData?.eligible ?? false; + const ocvElegible = ocvData?.elegible ?? false; logger.info(`OCV data: ${JSON.stringify(ocvData)}`); - logger.info(`Proposal ${proposal.id} should move to DELIBERATION. Approval count: ${approvalCount}, min approvals: ${this.config.considerationPhase.minReviewerApprovals}, OCV eligible: ${ocvEligible}`); + logger.info(`Proposal ${proposal.id} should move to DELIBERATION. Approval count: ${approvalCount}, min approvals: ${this.config.considerationPhase.minReviewerApprovals}, OCV elegible: ${ocvElegible}`); - return isMinApprovals || ocvEligible; + return isMinApprovals || ocvElegible; } private async shouldMoveBackToConsideration(proposal: ProposalWithVotes): Promise { const approvalCount = this.countValidApprovals(proposal); const isMinApprovals = approvalCount >= this.config.considerationPhase.minReviewerApprovals; const ocvData = proposal.OCVConsiderationVote?.voteData as OCVVoteResponse | undefined; - const ocvEligible = ocvData?.eligible ?? false; + const ocvElegible = ocvData?.elegible ?? false; logger.info(`OCV data: ${JSON.stringify(ocvData)}`); - logger.info(`Proposal ${proposal.id} should move back to CONSIDERATION. Approval count: ${approvalCount}, min approvals: ${this.config.considerationPhase.minReviewerApprovals}, OCV eligible: ${ocvEligible}`); + logger.info(`Proposal ${proposal.id} should move back to CONSIDERATION. Approval count: ${approvalCount}, min approvals: ${this.config.considerationPhase.minReviewerApprovals}, OCV elegible: ${ocvElegible}`); - return !isMinApprovals && !ocvEligible; + return !isMinApprovals && !ocvElegible; } private countValidApprovals(proposal: ProposalWithVotes): number { diff --git a/src/tasks/ocv-vote-counting.ts b/src/tasks/ocv-vote-counting.ts index 473d15a..4ad18a3 100644 --- a/src/tasks/ocv-vote-counting.ts +++ b/src/tasks/ocv-vote-counting.ts @@ -221,7 +221,7 @@ async function processProposals() { if (moveResult) { const voteInfo: ProposalVoteInfo = { - ocv_eligible: moveResult.ocvEligible, + ocv_eligible: moveResult.ocvElegible, reviewer_votes_given: moveResult.reviewerVotesGiven, reviewer_votes_required: moveResult.reviewerVotesRequired };