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
[TIMOB-25468] Android: Always include 'source' property with events #9643
Conversation
Generated by 🚫 dangerJS |
Question: |
@garymathews @jquick-axway Going with this one or a different solution? We're trying code-freeze for 7.0.1 on Monday :-). |
598b3ce
to
73c257d
Compare
@jquick-axway Yep, updated PR. |
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.
CR: Pass
@garymathews , When I run the test code on pixel android 7.1.1 I do not get any alert with source info. Android 6.0.1 device is fine. Also android emulator 7.1.1 shows the info but android 8.0 emulator does not. This happens on 6.3.0.GA as well. I also checked on my samsung s8+ running android 7.1.1 & I get the same issue. Below are the logs for pixel 7.1.1 device:
|
@lokeshchdhry So it is a chromium issue, but it's of no concern. I receive the |
FR Passed. According to @garymathews the errors above are due to soasta & not related to this PR. There will be another PR for it.
Studio Ver: 5.0.0.201712081732 |
source
property in proxy eventsTEST CASE
JIRA Ticket