Feedback from updated version (V. 0.4)

[hpmaxi]

From Elena's feedback

• Add validation to amount fields, only valid amounts should pass validation example
• Avoid clearing out fields if the tx is not sent. Ex. Canceled submit modal, some error inside initiateAuction
• Show a modal if the tx is successful with the auctionId + clear out the fields on the form after successful transaction
• Divider between 4 and 5 field must be removed
• Dates fields are narrower.
• Change in tooltipText and labels buytAmount to amount to buy, allowListManager as List Manager, etc.
• Calendar is transparent, make it solid.
• Invalid ERC20 for 0x4dbcdf9b62e891a7cec5a2568c3f4faf9e8abe2b is incorrect.
• No validations when dates are empty or cancelDate > closingDate. Keep 'Build transaction' disabled when date = today + time in the past. Errors could being cleared out by the build button.
• Cancel TX should not reset form.
• gasPrice/gasLimit is not set automatically?
• ThirdParty errors in console.

[elena-zh]

Hi Maxi!
I think, that the next cases are already fixed:

• Invalid ERC20 for 0x4dbcdf9b62e891a7cec5a2568c3f4faf9e8abe2b is incorrect.
• Cancel TX should not reset form.

[hpmaxi]

@elena-zh
Third Party errors seems related to safe-react-app-sdk package. I can't solve it for the moment, and about the modal with the auctionId there is the issue #29 . Maybe we can close this?

[elena-zh]

Hi @hpmaxi ,
yes, we can close this issue, as for unfixed cases separate issues are created.

Thanks