[GitHub] [incubator-superset] ktmud commented on a change in pull request #10317: chore: type welcome

[GitBox <gi...@apache.org>]

ktmud commented on a change in pull request #10317:
URL: https://github.com/apache/incubator-superset/pull/10317#discussion_r454804319



##########
File path: superset-frontend/src/welcome/DashboardTable.tsx
##########
@@ -17,35 +17,45 @@
  * under the License.
  */
 import React from 'react';
-import PropTypes from 'prop-types';
 import { t } from '@superset-ui/translation';
 import { SupersetClient } from '@superset-ui/connection';
 import moment from 'moment';
 import { debounce } from 'lodash';
 import ListView from 'src/components/ListView/ListView';
 import withToasts from 'src/messageToasts/enhancers/withToasts';
+import { Dashboard } from 'src/types/bootstrapTypes';
+import { FetchDataConfig } from 'src/components/ListView/types';
 
 const PAGE_SIZE = 25;
 
-class DashboardTable extends React.PureComponent {
-  static propTypes = {
-    addDangerToast: PropTypes.func.isRequired,
-    search: PropTypes.string,
-  };
+interface DashboardTableProps {

Review comment:
       I'm not sure about this recommendation. I like how interfaces gives you an alias and really makes intellisense and error messages more readable. Even TypeScript's official doc uses Interface: https://www.typescriptlang.org/docs/handbook/jsx.html#function-component
   
   We have used interface in a lot of places and I have yet to see any problem.




----------------------------------------------------------------
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:
users@infra.apache.org



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