/builder/:id route needs to return 404 if it id ends up not existing. #484

rwblair · 2019-02-05T20:53:56Z

As of now it will just attempt to display the Overview tab. Users hopefully wont be putting garbage in that url, but if we hear from the api that the provided api we need to bail out of the builder component and display 404.tsx

…nalysis function accordingly. Fixes #555 Disable next button on contrast tab if no contrasts present. Never set flag to generate reports if no contrasts exist. Fixes #537 Have analyses names line break if too long. Fixes #562 Non existant analysis IDs will now redirect to /builder/. Fixes #484 on error codes 400 and above do not attempt to parse response as json, just return status code so component can figure out what it wants to do. Fixes #527

adelavega added bug frontend labels Feb 19, 2019

adelavega added this to To Do (Frontend) in 0.9 via automation Mar 27, 2019

rwblair moved this from To Do (Frontend) to In Progress in 0.9 May 14, 2019

rwblair mentioned this issue May 14, 2019

Fixes for frontend regressions. #566

Merged

adelavega closed this as completed in #566 May 15, 2019

0.9 automation moved this from In Progress to Done May 15, 2019

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

/builder/:id route needs to return 404 if it id ends up not existing. #484

/builder/:id route needs to return 404 if it id ends up not existing. #484

rwblair commented Feb 5, 2019

/builder/:id route needs to return 404 if it id ends up not existing. #484

/builder/:id route needs to return 404 if it id ends up not existing. #484

Comments

rwblair commented Feb 5, 2019