-
Notifications
You must be signed in to change notification settings - Fork 11
Setup interface for jobs #177
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
Conversation
✅ Deploy Preview for ami-storybook ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for ami-web ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
|
Wow! This was a big one. Really cool though. I love seeing the different type of status indicators in different stages and UI contexts. I am glad we have both the vertical and horizontal wizards to help break up complexity of our multi-step processes. The one thing I changed is the display text for the Jobs menu item. Even though we are creating & listing Job objects, I still like "Batch ID" for the section title. But I am open to debate that with the team! |
Ah yes, let's change that for now and revisit if the team disagrees! :) |
In this PR we setup interface for jobs and job details (using dummy data until BE is ready). We are still missing interface for setting up a new job, but this PR started to get huge! As part of these changes, we are also removing the previous "Batch ID" page.
Screenshots
Some of the new generic UI components that was added as part of creating the UI:

Table for jobs:

Job details:
