yardz commented on a change in pull request #13225:
URL: https://github.com/apache/superset/pull/13225#discussion_r578879975
##########
File path: superset-frontend/src/views/CRUD/welcome/ActivityTable.tsx
##########
@@ -178,7 +115,13 @@ export default function ActivityTable({
/>
<>
{activityData[activeChild]?.length > 0 ? (
- <ActivityContainer>{renderActivity()}</ActivityContainer>
+ <ActivityContainer>
+ <ActivityTableRow
Review comment:
I gave this name following the logic "table->row". If you look at
ActivityTableRow it receives an array and returns several "cards" inside a
Fragment.
I had thought of "ActivityList" (list of cards right?), but I chose to keep
it as "ActivityTableRow" as there are some improvements to be made in the props
and these components are very related...
----------------------------------------------------------------
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]
