Details
-
Improvement
-
Resolution: Fixed
-
High
-
None
-
None
-
-
Add-Ons 60
-
5
Description
I see that there are some spaces for improvements for this part of ABn Testing.
The event sending javascript is quite important because it's the integration point (calling to Magnolia REST endpoint) between the ABn Testing page and API gateway (where the incoming request process and store the events data)
- Therefore, let's check and see what we can improve in some aspect below:
- Code quality
- Performance and stability
- Error handling
- The sent event json has an empty value of the field `userId`, we noticed this issue while running traffic generator, and this is not only happened with traffic generator, also every first request in a new session (try to use a Firefox private window to check this).
Dev notes:
The related Js located in here: https://git.magnolia-cms.com/projects/ADDON/repos/ab-testing/browse/magnolia-abtesting/src/main/resources/ab-testing/webresources/js
- The main one is `user-actions-bundle.js`
Checklists
Acceptance criteria