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

Dashboard's tables update #834

Merged
merged 6 commits into from
Feb 9, 2023
Merged

Dashboard's tables update #834

merged 6 commits into from
Feb 9, 2023

Conversation

Ajmal
Copy link
Contributor

@Ajmal Ajmal commented Feb 8, 2023

Update for #829 feedback

  1. Standardize tables with label for:
  • Jobs
  • Batches
  • Crons
  1. Fix missing div tag

Screen Shot 2023-02-08 at 14 15 18
Screen Shot 2023-02-08 at 14 15 11
Screen Shot 2023-02-08 at 14 14 57

Screen Shot 2023-02-08 at 14 15 22

@bensheldon
Copy link
Owner

@Ajmal this is great! You have definitely surpassed my skills with flexbox 🙇🏻

It looks like the badges stopped being aligned to the end of the row. Do you think you can fix that?

Screenshot 2023-02-08 at 2 20 37 PM

@Ajmal
Copy link
Contributor Author

Ajmal commented Feb 9, 2023

@bensheldon sure, I fixed these tables

Jobs
Screenshot 2023-02-09 at 8 38 07 AM

Jobs/:id/Executions
Screenshot 2023-02-09 at 8 38 23 AM

Batches
Screenshot 2023-02-09 at 8 38 32 AM

Thank you

@bensheldon bensheldon added the refactor Code changes that do not introduce new features label Feb 9, 2023
@bensheldon bensheldon merged commit f7e2889 into bensheldon:main Feb 9, 2023
@bensheldon
Copy link
Owner

@Ajmal thank you! I really appreciate it. I'd love to know if you have any other ideas/improvements for the Dashboard or anything else about GoodJob. This is excellent.

@Ajmal
Copy link
Contributor Author

Ajmal commented Feb 9, 2023

@bensheldon sure, I have not use GoodJob in production environment yet. I will give feedback for Dashboard after I migrate from DJ to GoodJob. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor Code changes that do not introduce new features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants