jscheffl commented on code in PR #50100:
URL: https://github.com/apache/airflow/pull/50100#discussion_r2070785462
##########
airflow-core/src/airflow/ui/src/pages/Events/Events.tsx:
##########
@@ -41,6 +42,43 @@ const eventsColumn = (
skeletonWidth: 10,
},
},
+ {
+ accessorKey: "event",
+ enableSorting: true,
+ header: "Event",
+ meta: {
+ skeletonWidth: 10,
+ },
+ },
+ {
+ accessorKey: "owner",
+ enableSorting: true,
+ header: "User",
+ meta: {
+ skeletonWidth: 10,
+ },
+ },
+ {
+ accessorKey: "extra",
+ cell: ({ row: { original } }) => {
+ if (original.extra !== null) {
+ try {
+ const parsed = JSON.parse(original.extra) as Record<string, unknown>;
+
+ return <RenderedJsonField content={parsed} jsonProps={{ collapsed:
true }} />;
Review Comment:
One one side I understand that per default you want to have it collapsed...
on the other side if I am looking for a specific event it is also burdensome to
expand all results individually? No real opinion but might also be hard to use
if collapsed... but expanded per default is also not great... mhm...
--
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]
