[WC-3452] Add pusher module to the repo#2258
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
4cd3a12 to
56a23fa
Compare
This comment has been minimized.
This comment has been minimized.
56a23fa to
fd463f9
Compare
This comment has been minimized.
This comment has been minimized.
fd463f9 to
acfb4d9
Compare
acfb4d9 to
fab3036
Compare
This comment has been minimized.
This comment has been minimized.
fab3036 to
f95239d
Compare
This comment has been minimized.
This comment has been minimized.
4d1b69c to
9429277
Compare
This comment has been minimized.
This comment has been minimized.
9429277 to
805d167
Compare
805d167 to
90ed6d3
Compare
AI Code Review
What was reviewed
Skipped (out of scope):
Findings🔶 Medium — pusher-web CHANGELOG not updated for breaking API changeFile: ## [Unreleased]
### Changed
- Replaced single `notifyActionName`/`notifyEventAction` properties with a configurable
`eventHandlers` list, allowing multiple Pusher events to be bound to separate client actions.
### Added
- Design-time validation: Studio Pro now reports an error when duplicate action names are configured.Run 🔶 Medium — Unit tests are a stub; changed behaviour has no coverageFile:
|
This adds pusher module to the repo, has old widget as an asses to copy into the module.