Skip to content
Snippets Groups Projects
Verified Commit c4354c13 authored by Jonathan Weth's avatar Jonathan Weth :keyboard:
Browse files

Fix celery progres page to show error messages on failure

parent 6238df53
No related branches found
No related tags found
3 merge requests!1237Release 3.0,!1226Resolve "Progress page for Celery tasks doesn't show error messages if task aborts with an exception",!1183Release 3.0
Pipeline #124810 failed
......@@ -38,6 +38,7 @@ Fixed
* When the Celery worker wasn't able to execute all tasks in time, notifications were sent multiple times.
* Changing the maintenance mode state spawned another SPA instance in the iframe
* Phone numbers couldn't be in regional format.
* Progress page didn't show error message on failure.
`3.0b3`_ - 2023-03-19
---------------------
......
......@@ -33,7 +33,7 @@
</message-box>
</div>
<message-box
v-if="progress.state === 'ERROR'"
v-if="progress.state === 'FAILURE'"
dense
type="error"
transition="slide-x-transition"
......@@ -60,13 +60,13 @@
<v-card-actions
v-if="
progress &&
(progress.state === 'ERROR' || progress.state === 'SUCCESS')
(progress.state === 'FAILURE' || progress.state === 'SUCCESS')
"
>
<back-button :href="progress.meta.backUrl" text />
<v-spacer />
<v-btn
v-if="progress.meta.additionalButton"
v-if="progress.meta.additionalButton && progress.state === 'SUCCESS'"
:href="progress.meta.additionalButton.url"
text
color="primary"
......@@ -108,7 +108,7 @@ export default {
},
watch: {
state(newState) {
if (newState === "SUCCESS" || newState === "ERROR") {
if (newState === "SUCCESS" || newState === "FAILURE") {
this.$apollo.queries.celeryProgressByTaskId.stopPolling();
this.$apollo.mutate({
mutation: gqlCeleryProgressFetched,
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment