brucearctor commented on code in PR #350:
URL:
https://github.com/apache/incubator-flagon-useralejs/pull/350#discussion_r1152330564
##########
example/index.js:
##########
@@ -34,28 +34,26 @@ window.userale.options({
* Note that for surgical filters, you may need to clear or reset back to a
global filter callback
* the same is true for the 'map' API. See examples below:
*/
-window.userale.filter(function (log) {
- var type_array = ['mouseup', 'mouseover', 'mousedown', 'keydown',
'dblclick', 'blur', 'focus', 'input', 'wheel'];
- var logType_array = ['interval'];
- return !type_array.includes(log.type) &&
!logType_array.includes(log.logType);
+window.userale.addCallbacks({
+ filter(log) {
+ var type_array = ['mouseup', 'mouseover', 'mousedown', 'keydown',
'dblclick', 'blur', 'focus', 'input', 'wheel'];
+ var logType_array = ['interval'];
+ return !type_array.includes(log.type) &&
!logType_array.includes(log.logType);
+ }
});
/**Log Mapping API
*
* the 'map' API allows you to add or modify new fields to your logs
* this example works with the "Click Me!" button at the top of index.html
*/
-document.addEventListener('click', function(e){
- if (e.target.innerHTML === 'Click Me!') {
- window.userale.map(function (log) {
+window.userale.addCallbacks({
+ map(log, e) {
+ if(e && e.type === 'click' && e.target.innerHTML === 'Click Me!') {
Review Comment:
Code cleanup is one thing to change ... but don't forget about potential
users. If a log message will not be produced, how likely is that to effect
consumers? I don't think that seems to be something we guarantee, but could be
considered a 'breaking' change, so things like that warrant at least
consideration [ OK to change, but important to consider ].
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@flagon.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
