Add maximum time info under spinner

aquiladev / remix-mythx-plugin

The plugin allows executing security check via MythX in Remix IDE

http://remythx.xyz

MIT License

16 stars 14 forks source link

Add maximum time info under spinner #17

Closed Gtonizuka closed 5 years ago

Gtonizuka commented 5 years ago

After user click on analyse it would be nice to see a message that says "we are analysing your contract. This should take up to X minutes" where X is the number of minutes the analysis process should take from the backend. This will be a bit more user friendly as the user will know that is not a very fast process the analysis.

aquiladev commented 5 years ago

is there a way to define timeouts in mythxjs?

aquiladev commented 5 years ago

Discord discussion:

sergii.bomko 07-01-19 at 3:03 PM
Sometimes I face an issue that request to MythX API hangs forever, it hangs the whole app. Are there any plans to add timeouts for analysis request?

Nathan 07-01-19 at 3:48 PM
@sergii.bomko There is not currently a way to add timeouts, but there are now plans to do so, hopefully it will get implemented soon. :slight_smile:

aquiladev commented 5 years ago

there is a separate icon with tooltip 'we are analyzing your contract. This should take up to 2 minutes' while analyzing

aquiladev commented 5 years ago

Request to show test on how long analysis will take instead of the button.

Options:

Annotation 2019-07-15 161019

Annotation 2019-07-15 161256