michael-s-molina opened a new pull request, #38690: URL: https://github.com/apache/superset/pull/38690
### SUMMARY `superset-extensions init` was not using the provided `name` but the `id` for generating the extension folder like `acme.example`. This caused confusion as the publisher was being added to the folder name and the output was not matching what the user typed. It also violated the following check: ``` Extension name must start with a letter and contain only lowercase letters, numbers, and hyphens (e.g., 'dashboard-widgets') ``` We don't need the publisher in the extension folder name to resolve collisions. Everything we need is inside `extension.json`. Users can still chose to put the publisher name if they wish to like 'acme-example'. ### TESTING INSTRUCTIONS 1. Run `superset-extensions init` 2. Use `example` as the extension name 3. Use `acme` as the publisher 4. Check that the extension folder name is `example` and not `acme.example` ### ADDITIONAL INFORMATION <!--- Check any relevant boxes with "x" --> <!--- HINT: Include "Fixes #nnn" if you are fixing an existing issue --> - [ ] Has associated issue: - [ ] Required feature flags: - [ ] Changes UI - [ ] Includes DB Migration (follow approval process in [SIP-59](https://github.com/apache/superset/issues/13351)) - [ ] Migration is atomic, supports rollback & is backwards-compatible - [ ] Confirm DB migration upgrade and downgrade tested - [ ] Runtime estimates and downtime expectations provided - [ ] Introduces new feature or API - [ ] Removes existing feature or API -- 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]
