Skip to content
This repository has been archived by the owner on Apr 6, 2020. It is now read-only.

Deploy Contract and Invoke Contract dialogs: counter-intuitive #39

Open
mwherman2000 opened this issue Feb 12, 2018 · 1 comment
Open

Comments

@mwherman2000
Copy link
Contributor

mwherman2000 commented Feb 12, 2018

You can call it a usability or functionality or workflow issue but it appears that the contract deployment transaction doesn't happen (execute) until you click the Test button on the Invoke Contract dialog. This is counter-intuitive and will lead to a lot of "user error". Checkout the function btnTestScript_Click() here:

private void btnTestScript_Click(object sender, EventArgs e)

People are going to expect the deployment should take place when you click the Deploy button in the Deploy Contract dialog. This should be refactored.

@ghost
Copy link

ghost commented Feb 21, 2018

Agreed, but this is the way it has been implemented in neo-project/neo-gui. For the sake of future updates I don't think it is a big enough problem to warrant changing the workflow.

Thoughts?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant