Migrate all DAG details to existing grid view dag details panel

[bbovenzi]

• Add any important information in /dags/{dag_id}/details to the dag details panel in grid view (when a user doesn't have a task or run selected).
• Redirect all urls
• Delete old dag details page

[Adaverse]

Can I take this up?

[potiuk]

Sure

[Adaverse]

Hi @bbovenzi,

There are information missing from the /dags/{dag_id}/details API, especially the ones in the DagModel debug information in current (old) details. Shall we add this info to the API or shall this be taken up separately? Some part of it has been discussed in this PR

Update:
Combining /dags/{dag_id} and /dags/{dag_id}/details gives almost all the information. Below is what is missing -

parent_dag
is_paused_at_creation
metadata
processor_subdir
registry
safe_dag_id

I think we can proceed. We can add the above whenever required anyways. Let me know your thoughts.

[bbovenzi]

Yeah, let's go forward with what we have. We can add those in later.

[Adaverse]

Is it fine to modify this this function in views.py to return json the way UI expects @bbovenzi?

[bbovenzi]

Is it fine to modify this this function in views.py to return json the way UI expects @bbovenzi?

Sure! I think theres a few places where we pass response-type: application/json for that exact reason. Are you trying to get info that the REST API does not have?

[Adaverse]

Yes! Still in double mind. Have already raised issue on it. So in future we will get all the data through api itself. So I think best way would be to first show all the data that's coming from dag/dag_id and have a button which on pressing will show all data that's there in the details endpoint but not in the dags endpoint. Hence even if in future the details returns more data that will be incorporated automatically without any tweaks in the UI.