search

DalgoT4D / webapp

GNU Affero General Public License v3.0
5 stars 35 forks source link

Error messaging on data statistics #941

Closed nairabhishek73 closed 5 days ago

nairabhishek73 commented 2 weeks ago

https://jam.dev/c/dcc79707-8ef0-4a31-bc6b-d631583c7569

In what situation would the try again button work? In the video you'll otice that I have no info on the problem and all i'm left to do is to keep hitting a button and hoping it works.

Ideally the user should be informed of the error and if try again doesn't work then be directed to support.

Moreover what is the purpose of the try again button? Do we expect individual rows to fail to load? If try again is not expected to change anything then just do away with that button. they can click away and click on the table again, else they can hit refresh for the full statistics.

Ishankoradia commented 2 weeks ago

Refresh does the same thing as try again but for all columns. Basically re-computing the statistics if some results are missing.

We will remove that button. We will need to display some text there when it fails to fetch, i was thinking maybe "Hit refresh to try again or reach out to ...." ?. @nairabhishek73

nairabhishek73 commented 1 week ago

Yes that works. Instead of asking them to reach out to us can we make this a bit easier for them so that they hit a button and we're automatically notified of the problem?

Ishankoradia commented 1 week ago

@nairabhishek73 doesnt make sense to add customer support just for this. If we are adding a button that notifies us of the issue, it should be for entire Dalgo platform.

We can tackle this a Dalgo feature instead of just data insights feature

For now we will just change the wording and do away with the button

mdshamoon commented 1 week ago

This is fixed. @nairabhishek73 ready for testing