villebro commented on a change in pull request #18027: URL: https://github.com/apache/superset/pull/18027#discussion_r784686914
########## File path: superset-frontend/packages/generator-superset/generators/plugin-chart/templates/README.erb ########## @@ -1,52 +1,40 @@ -## superset-plugin-chart-<%= packageName %> +# <%= packageName %> -This plugin provides <%= description %> for Superset. +This is the <%= description %>. ### Usage -Configure `key`, which can be any `string`, and register the plugin. This `key` will be used to lookup this chart throughout the app. +To build the plugin, run the following commands: -```js -import <%= packageLabel %>ChartPlugin from '@superset-ui/plugin-chart-<%= packageName %>'; +``` +npm i --force +npm run build +``` + +To run the plugin in development mode (=rebuilding whenever changes are made), start the dev server with the following command: -new <%= packageLabel %>ChartPlugin() - .configure({ key: '<%= packageName %>' }) - .register(); +``` +npm run dev +``` + +To add the package to Superset, go to the `superset-frontend` subdirectory in your Superset source folder (assuming both the `superset-plugin-chart-liquid` plugin and `superset` repos are in the same root directory) and run Review comment: thanks for catching! It's an example plugin that I've worked on while testing this template -- 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