fix(GNS): use the correct token amount when curation or owner tax are zero

[pcarranzav]

Before this fix, if curation or owner tax were zero, GNS would try to mint signal for the new version with zero tokens, which would revert.

[openzeppelin-code]

fix(GNS): use the correct token amount when curation or owner tax are zero

Generated at commit: 322658ed487c41db3a8dd33370c8392998b0671b

🚨 Vulnerabilities Summary

Process	Issues	Results
Contract Inspector	note low critical high Total	32 18 1 1 52
Dependency Checker	low note Total	1 23 24

---

For more details view the full report in OpenZeppelin Code

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (cfff549) 92.60% compared to head (322658e) 92.65%.
Report is 13 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #873      +/-   ##
==========================================
+ Coverage   92.60%   92.65%   +0.04%     
==========================================
  Files          47       47              
  Lines        2368     2368              
  Branches      432      432              
==========================================
+ Hits         2193     2194       +1     
+ Misses        175      174       -1     

Flag	Coverage Δ
unittests	92.65% <100.00%> (+0.04%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[tmigone]

Should we also test it doesn't revert with _curationTaxPercentage = 0?

[pcarranzav]

Should we also test it doesn't revert with _curationTaxPercentage = 0?

Good point, created #891 to address in a separate PR