[MGNLHOOK-141] Integrate webhooks with Norsu Created: 13/Jun/22 Updated: 05/Oct/22 Resolved: 05/Oct/22 |
|
| Status: | Closed |
| Project: | Magnolia Webhooks |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 2.0.0 |
| Type: | Story | Priority: | Neutral |
| Reporter: | Jaroslav Simak | Assignee: | Dai Ha |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Σ Remaining Estimate: | Not Specified | Remaining Estimate: | Not Specified |
| Σ Time Spent: | Not Specified | Time Spent: | Not Specified |
| Σ Original Estimate: | Not Specified | Original Estimate: | Not Specified |
| Issue Links: |
|
|||||||||||||||||||||||||
| Sub-Tasks: |
|
|||||||||||||||||||||||||
| Team: | ||||||||||||||||||||||||||
| Documentation update required: |
Yes
|
|||||||||||||||||||||||||
| Date of First Response: | ||||||||||||||||||||||||||
| Epic Link: | Webhook on SaaS | |||||||||||||||||||||||||
| Sprint: | DevX 20 | |||||||||||||||||||||||||
| Story Points: | 5 | |||||||||||||||||||||||||
| Description |
|
Take into account two things:
|
| Comments |
| Comment by Javier Benito [ 26/Aug/22 ] |
|
It sounds good to me, instead of having a Node inside the event that is sent to the queue, have a different object less JCR-related, or maybe just the properties that we are using right now. As far as I know, the node inside the event is not used nor propagated. We should also change probably the way it's retrieved, now it's done using getJCRNode which is available on a RuleBasedCommand. |
| Comment by Jaroslav Simak [ 26/Aug/22 ] |
|
Removal the JCR API will be handled by |
| Comment by Javier Benito [ 26/Aug/22 ] |
|
Perfect, should we add it to |
| Comment by Jaroslav Simak [ 26/Aug/22 ] |
|
Yes please |
| Comment by Javier Benito [ 26/Aug/22 ] |
|
Done |