Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add job retry panel to under job information #1397

Closed
wants to merge 1 commit into from

Conversation

CharlesBilbo
Copy link

This pull request relocates the retry widget shifting it from the bottom of the page to a more prominent position beneath the job information section. While seemingly minor, this adjustment addresses a significant issue. Myself and several colleagues, were unaware of its existence due to its placement at the page's bottom. Particularly with larger payloads, it was effectively concealed.

image

@CharlesBilbo
Copy link
Author

I wasnt sure if i needed to recompile the app.js or if thats done when the code is merged

@lukasyelle
Copy link

Definitely would find this useful! 👍

@driesvints
Copy link
Member

I wasnt sure if i needed to recompile the app.js or if thats done when the code is merged

That's all done automatically.

@taylorotwell
Copy link
Member

Doesn't this just mean now that the sections you moved to the bottom would be "undiscovered"?

@CharlesBilbo
Copy link
Author

not really since those components are present at all time while this one only appears when the button is clicks and is easily missed since there's not an indication that the job was retried.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants