mfyuce commented on a change in pull request #11234:
URL: https://github.com/apache/superset/pull/11234#discussion_r629709595
##########
File path: superset/models/dashboard.py
##########
@@ -145,7 +154,7 @@ class Dashboard( # pylint:
disable=too-many-instance-attributes
]
def __repr__(self) -> str:
- return self.dashboard_title or str(self.id)
+ return f"Dashboard<{self.slug or self.id}>"
Review comment:
After this change, when adding a dashboard schedule, dashboard names are
seen as "Dashboard <1>" etc... with no name at all and hard to select a
dashboard for a schedule from its ID.... Is not there a better option other
than this?
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
