-
Notifications
You must be signed in to change notification settings - Fork 7
Cleanup and Refactor #175
Copy link
Copy link
Open
Labels
Description
Task description
- Refactor MediaSessionManager to accept edgeModule and configurationModule as a constructor parameter and update mediaModule and mediaSession accordingly (Unified Edge network queue and Added response event dispatch and handling #173 (comment))
- Cleanup media helper functions (Unified Edge network queue and Added response event dispatch and handling #173 (comment))
- Update the internal structure of Media API request event fields (xdmData, data, etc..)
- Extract repeating logic to callback helper (Add setConsent public API and unit tests #171 (comment))
- Cleanup eventProcessor._sendEvent(). Pass the entire event object to edgeModule.processEvent() instead of requestId, data and timestamp individually.
- refactor buildEdgeURL (Implement Extracting & Caching State:Store and LocationHint from Konductor responses and Attaching to subsequent requests #177 (comment)
- Standardize init for all modules. Add init() method to all and follow the pattern across.
- Update IdentityModule to use EdgeModule to send getEcid request*
- Update tests based on refactor
- Add try catch to all module classes
CreateMediaSession:
' TODO: The playhead value in XDM is an integer type. if it allows int-64, we should change the type to longInteger.
startNewSession: function(playhead as longinteger) as string
* might not do
Additional implementation details or code snippet(s)
No response
Reactions are currently unavailable