Agoric / dapp-econ-gov

Governance UI for Inter Protocol economic committee
0 stars 3 forks source link

Update Econ-Gov UI to handle + validate new git hash input format #109

Closed otoole-brendan closed 1 week ago

otoole-brendan commented 2 months ago

The Econ Gov UI can take any string that the EC pastes into the field (which is subsequently voted in by EC as the endorsed UI - to become the latest UI version on chain).

We should add client-side validation so it helps guide them with the expected value, ensuring it's in the form of a valid git url, or ipfs hash for backward compatibility

otoole-brendan commented 1 month ago

Updated to reflect no contract update required. Change is only client-side for validation

LuqiPan commented 1 month ago

Is this absolutely necessary or a nice to have? Can we solve this with some other way, i.e. documentations, educations, etc?

samsiegart commented 1 month ago

It's not necessary, it's a nice-to-have. Yes, documentation/personal guidance for the EC would be warranted either way.