-
-
Notifications
You must be signed in to change notification settings - Fork 14
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
Feat/workflow list enhancement #305
base: dev
Are you sure you want to change the base?
Conversation
@tdari this is looking neat, I like compact! we are missing the "visualize workflow" action, though. Or, in your proposed design, how does the user navigate to the specific workflow visualization page? |
hi, @luiztauffer. Individual list items are still redirecting to the details page. It's working as it was before. |
@vinicvaz why do we have this restriction here in
|
Hey @tdari . In Airflow, if a DAG has a start date set in the future, triggering a run will only schedule it for execution after the start date, transitioning the DAG to a 'scheduled' state instead of running it immediately. That is why we are blocking this operation for workflows with start date in the future. |
I see thanks @vinicvaz |
1ebf8bb
to
959cd5d
Compare
I've finished most of this PR. Can you test it before I start writing tests? There may be some mistakes I haven't noticed, and I've had to decide on naming conventions, endpoints, etc. Let me know if you have a better proposal. I couldn't separate commits as I expected. Sorry for such a big change list. |
959cd5d
to
65f90b4
Compare
This PR will enhance the workflow list by adding checkboxes to control workflows and changing the UI for this new feature.
Add a control for the disabled state of the action buttons.Current UI (WIP):