search

Project-OSmOSE / osmose-app

PAM annotation platform
https://osmose.ifremer.fr/app/
GNU General Public License v3.0
5 stars 5 forks source link

Improve spectro loading error handling #111

Open ixio opened 1 year ago

ixio commented 1 year ago

Currently if you set the zoom level of a SpectroConfig to a higher level than the images available (if it's created for zoom level 2 with 1+2+4 tiles then moving it to zoom level 3 will allow to try and load the 8tiles level that doesn't exist).

When trying to load a non-existing zoom level the 404 errors make the app crash. It should have smoother error handling, like maybe just an alert saying failed to load 8x zoom and staying at the last working zoom level. The exact message an whether it uses alert/popup/popin should be seen with research team.

ElodieENSTA commented 9 months ago

Show appropriate message for 404 on wav or png loading (even if not due to extra-zoom)

ElodieENSTA commented 8 months ago

Show appopriate message when audio loading has 403 error (forbidden) -> probably due to insufficient level (chmod 777)

ElodieENSTA commented 7 months ago

Since https://github.com/Project-OSmOSE/osmose-app/releases/tag/1.8.0 the application is not "crashing" anymore. Just need a proper error message