search

wearefrank / ladybug-frontend

Frontend for Ladybug
Apache License 2.0
1 stars 7 forks source link

Make use of delete all in report api #370

Closed jacodg closed 2 months ago

jacodg commented 6 months ago

See https://github.com/wearefrank/ladybug/pull/218

Can we use this function when using the delete button without having a report selected?

The delete all needs to have an extra "are you sure?" action.

mhdirkse commented 2 months ago

Dit lijkt me error prone. Stel dat de GUI en de server niet gesynchroniseerd zijn. De gebruiker wil alle rapporten weggooien die hij ziet. Onze frontend code stuurt de API request om alle rapporten weg te gooien van de storage op de server. Maar er verdwijnen andere rapporten dan de gebruiker denkt. Dit probleem geldt zowel voor de debug tab als voor de test tab. Eens, @MatthijsSmets ?

jacodg commented 2 months ago

Waarom zouden er andere rapporten verdwijnen dan de gebruiker denkt? Als hij alle reports van een view/storage weg wil gooien en de "weet je het zeker?" bevestigd heeft waarom zouden er dan reports van een andere storage weggegooid worden?

MatthijsSmets commented 2 months ago

I agree with @jacodg, adding an Are you sure? popup should be enough