-
Notifications
You must be signed in to change notification settings - Fork 7
Closed
Description
Remove deprecated store exports and update RFC documentation.
Ref #320
Note: React cleanup was completed in Phase 3 (#367, PR #372) since the React Player API needed to replace the old context/createStore exports.
Note: Lit cleanup was completed in Phase 4 (#368, PR #374) since the HTML Player API needed to replace the old createStore/mixins exports.
Scope
| Component | Description |
|---|---|
store/react |
✅ Done in PR #372 |
store/lit |
✅ Done in PR #374 |
| Update RFC | Reflect final architecture with Store v2, PlayerTarget, selectors |
Removed Exports
From @videojs/store/react: ✅ Done in PR #372
createStoreuseStoreContextStoreContextProviderCreateStoreConfigCreateStoreResultProviderProps
From @videojs/store/lit: ✅ Done in PR #374
createStorecreateProviderMixincreateContainerMixincreateStoreMixinCreateStoreConfigCreateStoreResultCreateStoreHostcontextKeyStoreConsumerStoreProvider
Files
✅ Done in PR feat(react): setup react player api #372packages/store/src/react/(delete deprecated, update index)✅ Done in PR feat(html): setup player api #374packages/store/src/lit/(delete deprecated, update index)rfc/player-api/*.md(update)
Acceptance Criteria
- Deprecated exports removed from
store/react(PR feat(react): setup react player api #372) - Deprecated exports removed from
store/lit(PR feat(html): setup player api #374) -
store/reactexports onlyuseStore,useSelector(PR feat(react): setup react player api #372) -
store/litexports onlyStoreController,SubscriptionController,StoreAccessor(PR feat(html): setup player api #374) - RFC updated to reflect final architecture
- All tests pass
- Build succeeds
SP: 2
Metadata
Metadata
Assignees
Labels
No labels
Type
Projects
Status
Done