Skip to content

Commit

Permalink
STUD-78-Consolidate-Distribute-Minting-form-logic-for-uploaded-songs (#…
Browse files Browse the repository at this point in the history
…689)

* refactor: Remove alerts that will never be visible

* fix: Convert owner percent to Number for yup test

Fix edge case where owner percentage is loaded into text input field
and value is read as String for the yup validation test.

* refactor: Remove unnecessary alert handlers

* refactor: Remove minting form step never reached

* refactor: Clean up remaining minting logic
  • Loading branch information
dmkirshon authored Jun 27, 2024
1 parent e5c4a8e commit 3bb37ae
Show file tree
Hide file tree
Showing 3 changed files with 96 additions and 311 deletions.
2 changes: 1 addition & 1 deletion apps/studio/src/common/formUtils.ts
Original file line number Diff line number Diff line change
Expand Up @@ -211,7 +211,7 @@ export const commonYupValidation = {
if (!owners) return false;

const percentageSum = owners.reduce((sum, owner) => {
return sum + owner.percentage;
return sum + Number(owner.percentage);
}, 0);

return percentageSum === 100;
Expand Down
Loading

0 comments on commit 3bb37ae

Please sign in to comment.