web
web copied to clipboard
Introduce more extension points and delete `scopes`
Description
This PR introduces more extension points, allows registering extension points through app bootstrapping and gets rid of the scopes property on the Extension type.
All of the changes are "under the hood", if anything is different in the UI now it's a bug in the PR.
Motivation and Context
Offer extension points throughout our platform so that extension developers can easily hook their extensions into existing extension points.
Types of changes
- [ ] Bug fix (non-breaking change which fixes an issue)
- [x] New feature (non-breaking change which adds functionality)
- [x] Breaking change (fix or feature that would cause existing functionality to change)
- [ ] Technical debt
- [ ] Tests
- [ ] Documentation
- [ ] Maintenance (e.g. dependency updates or tooling)
Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.
@dschmidt @JammingBen ready for a thorough re-review. As far as I can tell the last remaining open question is whether the files app extension points should be defined and registered via the files app or defined in web-pkg and registered via the runtime... I added all extension point ids to the dev docs, so we might as well say that that's the source of truth for existing extension point ids. But might also be a good idea to have the extension point objects available in other apps. What do you think?
Quality Gate passed
Issues
1 New issue
0 Accepted issues
Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code