feat(dashboard): Add metadata bar to the header

superset-frontend/spec/fixtures/mockDashboardInfo.js

@@ -31,6 +31,11 @@ export default {
       },
     ],
   },
+  changed_on_delta_humanized: '7 minutes ago',
+  changed_by_name: 'John Doe',
+  created_on_delta_humanized: '10 days ago',
+  created_by_name: 'Kay Mon',
+  owners: [{ first_name: 'John', last_name: 'Doe', id: 1 }],
   userId: 'mock_user_id',
   dash_edit_perm: true,
   dash_save_perm: true,

superset-frontend/src/dashboard/components/Header/Header.test.tsx

@@ -44,6 +44,11 @@ const createProps = () => ({
         ],
       },
     },
+    changed_on_delta_humanized: '7 minutes ago',
+    changed_by_name: 'John Doe',
+    created_on_delta_humanized: '10 days ago',
+    created_by_name: 'Kay Mon',
+    owners: [{ first_name: 'John', last_name: 'Doe', id: 1 }],
   },
   user: {
     createdOn: '2021-04-27T18:12:38.952304',
@@ -187,6 +192,17 @@ test('should publish', () => {
   expect(mockedProps.savePublished).toHaveBeenCalledTimes(1);
 });
 
+test('should render metadata', () => {
+  const mockedProps = createProps();
+  setup(mockedProps);
+  expect(
+    screen.getByText(mockedProps.dashboardInfo.created_by_name),
+  ).toBeInTheDocument();
+  expect(
+    screen.getByText(mockedProps.dashboardInfo.changed_on_delta_humanized),
+  ).toBeInTheDocument();
+});
+
 test('should render the "Undo" action as disabled', () => {
   setup(editableProps);
   expect(screen.getByTestId('undo-action').parentElement).toBeDisabled();

superset-frontend/src/dashboard/components/Header/index.jsx

@@ -46,6 +46,7 @@ import PublishedStatus from 'src/dashboard/components/PublishedStatus';
 import UndoRedoKeyListeners from 'src/dashboard/components/UndoRedoKeyListeners';
 import PropertiesModal from 'src/dashboard/components/PropertiesModal';
 import { chartPropShape } from 'src/dashboard/util/propShapes';
+import getOwnerName from 'src/utils/getOwnerName';
 import {
   UNDO_LIMIT,
   SAVE_TYPE_OVERWRITE,
@@ -55,6 +56,7 @@ import setPeriodicRunner, {
   stopPeriodicRender,
 } from 'src/dashboard/util/setPeriodicRunner';
 import { PageHeaderWithActions } from 'src/components/PageHeaderWithActions';
+import MetadataBar, { MetadataType } from 'src/components/MetadataBar';
 import DashboardEmbedModal from '../EmbeddedModal';
 import OverwriteConfirm from '../OverwriteConfirm';
 
@@ -435,6 +437,26 @@ class Header extends React.PureComponent {
     this.setState({ showingEmbedModal: false });
   };
 
+  getMetadataItems = () => {
+    const { dashboardInfo } = this.props;
+    return [
+      {
+        type: MetadataType.LastModified,
+        value: dashboardInfo.changed_on_delta_humanized,
+        modifiedBy: dashboardInfo.changed_by_name || t('Not available'),
+      },
+      {
+        type: MetadataType.Owner,
+        createdBy: dashboardInfo.created_by_name || t('Not available'),
+        owners:
+          dashboardInfo.owners.length > 0
+            ? dashboardInfo.owners.map(getOwnerName)
+            : t('None'),
+        createdOn: dashboardInfo.created_on_delta_humanized,
+      },
+    ];
+  };
+
   render() {
     const {
       dashboardTitle,
@@ -535,6 +557,12 @@ class Header extends React.PureComponent {
                 visible={!editMode}
               />
             ),
+            !editMode && (
+              <MetadataBar
+                items={this.getMetadataItems()}
+                tooltipPlacement="bottom"
+              />
+            ),
           ]}
           rightPanelAdditionalItems={
             <div className="button-container">

superset/dashboards/api.py

@@ -175,6 +175,7 @@ def ensure_thumbnails_enabled(self) -> Optional[Response]:
         "created_by.first_name",
         "created_by.id",
         "created_by.last_name",
+        "created_by_name",
         "dashboard_title",
         "owners.id",
         "owners.first_name",

superset/dashboards/schemas.py

@@ -187,13 +187,15 @@ class DashboardGetResponseSchema(Schema):
     certification_details = fields.String(
         metadata={"description": certification_details_description}
     )
+    created_by_name = fields.String()
     changed_by_name = fields.String()
     changed_by = fields.Nested(UserSchema(exclude=["username"]))
     changed_on = fields.DateTime()
     charts = fields.List(fields.String(metadata={"description": charts_description}))
     owners = fields.List(fields.Nested(UserSchema(exclude=["username"])))
     roles = fields.List(fields.Nested(RolesSchema))
     tags = fields.Nested(TagSchema, many=True)
+    created_on_humanized = fields.String(data_key="created_on_delta_humanized")
     changed_on_humanized = fields.String(data_key="changed_on_delta_humanized")
     is_managed_externally = fields.Boolean(allow_none=True, dump_default=False)
 

superset/models/dashboard.py

@@ -242,6 +242,12 @@ def changed_by_name(self) -> str:
             return ""
         return str(self.changed_by)
 
+    @property
+    def created_by_name(self) -> str:
+        if not self.created_by:
+            return ""
+        return str(self.created_by)
+
     @property
     def data(self) -> dict[str, Any]:
         positions = self.position_json

tests/integration_tests/dashboards/api_tests.py

@@ -399,6 +399,7 @@ def test_get_dashboard(self):
                     "first_name": "admin",
                     "last_name": "user",
                 },
+                "created_by_name": "admin user",
                 "id": dashboard.id,
                 "css": "",
                 "dashboard_title": "title",
@@ -423,11 +424,13 @@ def test_get_dashboard(self):
         data = json.loads(rv.data.decode("utf-8"))
         self.assertIn("changed_on", data["result"])
         self.assertIn("changed_on_delta_humanized", data["result"])
+        self.assertIn("created_on_delta_humanized", data["result"])
         for key, value in data["result"].items():
             # We can't assert timestamp values
             if key not in (
                 "changed_on",
                 "changed_on_delta_humanized",
+                "created_on_delta_humanized",
             ):
                 self.assertEqual(value, expected_result[key])
         # rollback changes