justinpark commented on code in PR #27857:
URL: https://github.com/apache/superset/pull/27857#discussion_r1594501119


##########
superset/models/dashboard.py:
##########
@@ -242,6 +242,12 @@ def changed_by_name(self) -> str:
             return ""
         return str(self.changed_by)
 
+    @property

Review Comment:
   @michael-s-molina This case is different. The property 'created_by' does not 
currently exist, therefore it is being added as a new property. The interface 
used for this addition is 'changed_by_name', which closely resembles the 
desired shape.



-- 
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: notifications-unsubscr...@superset.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org
For additional commands, e-mail: notifications-h...@superset.apache.org

Reply via email to