-
Notifications
You must be signed in to change notification settings - Fork 5.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Transfer events
and notifications
service refs from their plugins to the backend-plugin-api
package
#24851
Comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Context
Ref: #24493
Backend service references are inconsistently provided in core and plugin packages. We intend to use the permissions model, where the service reference resides in the
backend-plugin-api
, but the default service is implemented in the plugin package.Description
Similar to permissions, the "events" and "notifications" service references should be extracted to the "coreService" namespace in backend-plugin-api.
Note
There is a related pull request opened, please refer to avoid conflicts.
Acceptance criteria
deprecated.ts
file@deprecated
and contain a comment explaining their replacementsbackend-test-utils
package should provide a mock and a factory for all core services (e.g. see)The text was updated successfully, but these errors were encountered: