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


##########
superset-frontend/packages/superset-ui-core/src/chart/types/VizType.ts:
##########
@@ -54,6 +54,7 @@ export enum VizType {
   Step = 'echarts_timeseries_step',
   Sunburst = 'sunburst_v2',
   Table = 'table',
+  TableAgGrid = 'table_ag_grid',

Review Comment:
   @amaannawab923 The purpose of creating GridTable is to expand additional 
features such as those required in the current situation, along with version 
control, in order to reuse the added functionalities in other components that 
are in progress, as suggested in the 
[SIP](https://github.com/apache/superset/issues/27645). The parts of 
src/component/GridTable that cannot be imported in the superset viz plugin can 
be migrated from the existing grid-table to superset-core-ui, making it 
possible. If you need assistance with this, please let me know.



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to