-
-
Notifications
You must be signed in to change notification settings - Fork 891
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
RF: Moved Analytics to class #3222
Conversation
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm also getting following error on master
object_hash.js:1 Uncaught Error: Object argument required.
at u (object_hash.js:1:2030)
at t (object_hash.js:1:753)
at Function.sha1 (object_hash.js:1:1085)
at serial_backend.js:443:77
at Object.send_message (msp.js:323:17)
at processUid (serial_backend.js:440:9)
at serial_backend.js:435:9
at Object.send_message (msp.js:323:17)
at checkReportProblems (serial_backend.js:403:9)
at processBoardInfo (serial_backend.js:383:9)
at onOpenVirtual (serial_backend.js:325:5)
at Object.connectVirtual (serial.js:215:13)
at Object.connect (serial.js:33:18)
at HTMLAnchorElement.<anonymous> (serial_backend.js:75:32)
at HTMLAnchorElement.dispatch (jquery.min.js:2:43064)
at y.handle (jquery.min.js:2:41048)
u @ object_hash.js:1
t @ object_hash.js:1
sha1 @ object_hash.js:1
(anonymous) @ serial_backend.js:443
send_message @ msp.js:323
processUid @ serial_backend.js:440
(anonymous) @ serial_backend.js:435
send_message @ msp.js:323
checkReportProblems @ serial_backend.js:403
processBoardInfo @ serial_backend.js:383
onOpenVirtual @ serial_backend.js:325
connectVirtual @ serial.js:215
connect @ serial.js:33
(anonymous) @ serial_backend.js:75
dispatch @ jquery.min.js:2
y.handle @ jquery.min.js:2
At least in virtual mode. And assume this exists in this pr. Could we fix that while we are at it?
Is that on Receiver tab? I spotted this error, not related to analytics: |
Found it! |
AUTOMERGE: (FAIL)
|
This comment has been minimized.
This comment has been minimized.
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Do you want to test this code? Here you have an automated build: |
Encapsulating the Analytics in a class.