Lee-W commented on code in PR #57710:
URL: https://github.com/apache/airflow/pull/57710#discussion_r2501440495
##########
airflow-core/src/airflow/ui/src/pages/Asset/AssetLayout.tsx:
##########
@@ -128,6 +128,26 @@ export const AssetLayout = () => {
</PanelResizeHandle>
<Panel defaultSize={30} minSize={20}>
<Header asset={asset} />
+ {asset?.extra && Object.keys(asset.extra).length > 0 && (
+ <Box mt={3} mb={3} px={3}>
+ <Text fontWeight="bold" mb={2}>
+ {translate("assets:static_asset_metadata")}
Review Comment:
```suggestion
{translate("assets:additional_data")}
```
maybe let's try additional data? the reason why I don't want to use `extra`
is because of https://github.com/apache/airflow/issues/55200, but suggestion is
welcome. Naming is hard 🤯
--
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]
