pierrejeambrun commented on code in PR #64845: URL: https://github.com/apache/airflow/pull/64845#discussion_r3058872669
########## airflow-core/src/airflow/ui/openapi-gen/queries/queries.ts: ########## @@ -950,10 +958,11 @@ export const useTaskInstanceServiceGetMappedTaskInstance = <TData = Common.TaskI * @param data.limit * @param data.offset * @param data.orderBy Attributes to order by, multi criteria sort is supported. Prefix with `-` for descending order. Supported attributes: `id, state, duration, start_date, end_date, map_index, try_number, logical_date, run_after, data_interval_start, data_interval_end, rendered_map_index, operator, logical_date, run_after, data_interval_start, data_interval_end` -* @returns TaskInstanceCollectionResponse Successful Response +* @returns unknown Successful Response Review Comment: Yeah apparently that's because of the code generator. It's trying to resolve the $ref but fails with the discriminator union. But the actual typescript annotation is generated from a different path and correctly produces the union. It's a codegen limitation in the comment rendering, not in the type rendering. I don't think it's important since we don't use that doc. `TaskInstanceServiceGetTaskInstancesDefaultResponse` resolves to `TaskInstanceOffsetCollectionResponse | TaskInstanceCursorCollectionResponse` -- 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]
