Skip to content

Commit

Permalink
Merge pull request #1823 from IntersectMBO/fix/1733-typeerror-cannot-…
Browse files Browse the repository at this point in the history
…read-properties-of-null-reading-drep_deposit

fix(#1733): fix accessing drep deposit from possibly empty epoch params
  • Loading branch information
MSzalowski committed Aug 26, 2024
2 parents e48e960 + ef32b86 commit fe370fe
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 5 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,7 @@ changes.
### Fixed

- Fix typescript bug leading to runtime error when entering Governance Action details page via direct link [Issue 1801](https://github.com/IntersectMBO/govtool/issues/1801)
- Fix accessing missing epochParams drep_deposit [Issue 1733](https://github.com/IntersectMBO/govtool/issues/1733)

### Changed

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ export const RolesAndResponsibilities = ({
const { t } = useTranslation();
const { isMobile } = useScreenDimension();

const deposit = getItemFromLocalStorage(PROTOCOL_PARAMS_KEY);
const epochParams = getItemFromLocalStorage(PROTOCOL_PARAMS_KEY);

const onClickContinue = () => setStep(2);

Expand Down Expand Up @@ -51,7 +51,7 @@ export const RolesAndResponsibilities = ({
/>,
]}
i18nKey="registration.rolesAndResponsibilitiesDescription"
values={{ deposit: correctAdaFormat(deposit.drep_deposit) }}
values={{ deposit: correctAdaFormat(epochParams?.drep_deposit) }}
/>
</Typography>
<CenteredBoxBottomButtons
Expand Down
4 changes: 2 additions & 2 deletions govtool/frontend/src/context/wallet.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -651,14 +651,14 @@ const CardanoProvider = (props: Props) => {
// Create cert object using one Ada as the deposit
dRepRegCert = DrepRegistration.new_with_anchor(
dRepCred,
BigNum.from_str(`${epochParams.drep_deposit}`),
BigNum.from_str(`${epochParams?.drep_deposit}`),
anchor,
);
} else {
console.error(t("errors.notUsingAnchor"));
dRepRegCert = DrepRegistration.new(
dRepCred,
BigNum.from_str(`${epochParams.drep_deposit}`),
BigNum.from_str(`${epochParams?.drep_deposit}`),
);
}
return Certificate.new_drep_registration(dRepRegCert);
Expand Down
2 changes: 1 addition & 1 deletion govtool/frontend/src/pages/RegisterAsDirectVoter.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -131,7 +131,7 @@ export const RegisterAsDirectVoter = () => {
>
<Trans
i18nKey="directVoter.registerDescription"
values={{ deposit: correctAdaFormat(epochParams.drep_deposit) }}
values={{ deposit: correctAdaFormat(epochParams?.drep_deposit) }}
components={[
<Link
onClick={() => openInNewTab("https://sancho.network/")}
Expand Down

0 comments on commit fe370fe

Please sign in to comment.