ESIPFed / Geoweaver

boost data pipeline's tangibility, enhance research productivity, reduce work anxiety
https://geoweaver.dev
MIT License
81 stars 44 forks source link

Feature: Remove failed processes from history #556

Closed gokulprathin8 closed 3 months ago

VisheshSaluja commented 3 months ago

First time using Remove failed history button worked Then i created a new process and tried the button, but the prompt did not go away after clicking ok Then i got this console error when the promt finally stopped

Screenshot 2024-08-23 at 11 17 29 PM
gokulprathin8 commented 3 months ago

First time using Remove failed history button worked Then i created a new process and tried the button, but the prompt did not go away after clicking ok Then i got this console error when the promt finally stopped

Screenshot 2024-08-23 at 11 17 29 PM

added a length > 0 check, check if that fixes the issue?

VisheshSaluja commented 3 months ago

This is what happens, the delete works but the prompt keeps on appearing and i think that in the backend it deletes every process history the first time but because the prompt keeps on appearing it throws the console error

and this doesn't happen the first time i use this button on running Geoweaver. First time it works as expected.

ezgif com-video-to-gif-converter

ZihengSun commented 3 months ago

Should be fixed now