-
Notifications
You must be signed in to change notification settings - Fork 8.5k
Kill @kbn/observable #17034
Copy link
Copy link
Closed
Labels
Team:CorePlatform Core services: plugins, logging, config, saved objects, http, ES client, i18n, etc t//Platform Core services: plugins, logging, config, saved objects, http, ES client, i18n, etc t//
Metadata
Metadata
Assignees
Labels
Team:CorePlatform Core services: plugins, logging, config, saved objects, http, ES client, i18n, etc t//Platform Core services: plugins, logging, config, saved objects, http, ES client, i18n, etc t//
Type
Fields
Give feedbackNo fields configured for issues without a type.
Extract@kbn/observablefrom thenew-platformbranch and PR it tomaster. That way we can discuss the observable details/apis separate from the architectural changes, and we can start relying on our implementation before merging the rest of thenew-platformbranch.Moving forward, we will use
rxjsdirectly.Reasons
rxjs?rxjs.rxjs.Utility functions
rxjsisn't enough, we can write new functions as needed.Usage practices
Subjects, educate developers rather than creating a wrapper or some blockade against using the feature.