-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(extension): check for empty assets and next bundle coin id on add… #1157
fix(extension): check for empty assets and next bundle coin id on add… #1157
Conversation
Quality Gate passedIssues Measures |
@@ -54,7 +54,7 @@ export const Form = ({ | |||
|
|||
const { setNewOutput } = useOutputs(); | |||
|
|||
const isEmptyAssets = assetBalances?.size === 0; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
in case assetBalances
is undefined
it won't be === 0
@mchappell @tommayeliog please correct me if i'm missing something.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you've not missed anything, that's the correct check to implement 🚀
const bundleDisabled = spendableCoin | ||
? !getNextBundleCoinId(spendableCoin?.toString(), assetBalances, tokensUsed, assets, cardanoCoin)?.length | ||
: false; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
was it intended to set bundleDisabled
to false
if spendableCoin
is falsy
?
@mchappell @tommayeliog please correct me if i'm missing something.
Allure Report
smokeTests: ✅ test report for d1c89ea3
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice!
… bundle
Checklist
Proposed solution
Explain how does this PR solves the problem stated in JIRA ticket.
You can also enumerate different alternatives considered while approaching this task.
Testing
Describe here, how the new implementation can be tested.
Provide link or briefly describe User Acceptance Criteria/Tests that need to be met
Screenshots
Attach screenshots here if implementation involves some UI changes