mikamikasuki commented on code in PR #74426:
URL: https://github.com/apache/airflow/pull/74426#discussion_r4219897244
##########
airflow-core/src/airflow/ui/src/pages/DagsList/DagsList.tsx:
##########
@@ -236,11 +243,10 @@ export const DagsList = () => {
const recentTasks = useRecentTaskStateCounts(data?.dags);
- const columns = createColumns(translate, runStateContext, {
+ const columns = createColumns(translate, {
Review Comment:
This call passes the options object as createColumns' second argument, but
createColumns still declares RunStateCountsContext as argument 2 and
destructures GetColumnsParams from argument 3. The DAG list then throws “Cannot
destructure property 'multiTeam' of 'undefined' as it is undefined,” matching
the error page captured by the Chromium, WebKit, and Firefox E2E jobs. Could
you remove the obsolete context parameter or pass the arguments in the expected
positions?
---
Drafted-by: Codex (model identifier unavailable in this session) (no human
review before posting)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
