-
Notifications
You must be signed in to change notification settings - Fork 90
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
feat(Inji-436): add telemetry events for app triggered warnings #986
feat(Inji-436): add telemetry events for app triggered warnings #986
Conversation
…t event subtype Signed-off-by: PuBHARGAVI <[email protected]>
…unction Signed-off-by: PuBHARGAVI <[email protected]>
…on and create an object with telemetry constants Signed-off-by: PuBHARGAVI <[email protected]>
Signed-off-by: PuBHARGAVI <[email protected]>
Signed-off-by: PuBHARGAVI <[email protected]>
…ck binding_auth_failed error Signed-off-by: PuBHARGAVI <[email protected]>
Signed-off-by: PuBHARGAVI <[email protected]>
… modal Signed-off-by: PuBHARGAVI <[email protected]>
Signed-off-by: PuBHARGAVI <[email protected]>
… instead of splitting the error in tsx files Signed-off-by: PuBHARGAVI <[email protected]>
…all languages Signed-off-by: PuBHARGAVI <[email protected]>
Signed-off-by: PuBHARGAVI <[email protected]>
Signed-off-by: PuBHARGAVI <[email protected]>
…emetry-events-for-vc-activation-flow Signed-off-by: PuBHARGAVI <[email protected]>
…m machines common functions into separate file Signed-off-by: PuBHARGAVI <[email protected]>
Signed-off-by: PuBHARGAVI <[email protected]>
Signed-off-by: PuBHARGAVI <[email protected]>
…emetry-events-for-app-triggered-warnings Signed-off-by: PuBHARGAVI <[email protected]>
… file Signed-off-by: PuBHARGAVI <[email protected]>
App.tsx
Outdated
import {MessageOverlay} from './components/MessageOverlay'; | ||
import SecureKeystore from 'react-native-secure-keystore'; | ||
import {isHardwareKeystoreExists} from './shared/cryptoutil/cryptoUtil'; | ||
import i18n from './i18n'; | ||
import './shared/flipperConfig'; | ||
import {__AppId} from './shared/GlobalVariables'; |
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.
remove this import
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.
Done
Signed-off-by: PuBHARGAVI <[email protected]>
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.
LGTM
…p#986) * refactor(INJI-435): add enum for flow type,end event status & interact event subtype Signed-off-by: PuBHARGAVI <[email protected]> * fix(INJI-435): change flow type value to camel case in getEventType function Signed-off-by: PuBHARGAVI <[email protected]> * feat(INJI-435): add start,interact & impression events in vc activation and create an object with telemetry constants Signed-off-by: PuBHARGAVI <[email protected]> * feat(INJI-435): add end,error events in vc activation flow Signed-off-by: PuBHARGAVI <[email protected]> * feat(INJI-435): increase the pressable area of kebab icon Signed-off-by: PuBHARGAVI <[email protected]> * feat: generate impression event only after going into otp modal & track binding_auth_failed error Signed-off-by: PuBHARGAVI <[email protected]> * refactor(INJI-435): move sendImpression event from addVc modal to otp modal Signed-off-by: PuBHARGAVI <[email protected]> * fix(INJI-435): add missing impression event in issuer machine Signed-off-by: PuBHARGAVI <[email protected]> * refactor(INJI-435): return bindingAuthFailedError from state machines instead of splitting the error in tsx files Signed-off-by: PuBHARGAVI <[email protected]> * fix(INJI-435): add padding and fix the activation status overflow in all languages Signed-off-by: PuBHARGAVI <[email protected]> * refactor(INJI-435): change the telemetry sdk path in package.json Signed-off-by: PuBHARGAVI <[email protected]> * refactor(INJI-435): extract esignetMosipVcItem and existingMosipVcItem machines common functions into separate file Signed-off-by: PuBHARGAVI <[email protected]> * refactor(INJI-435): remove duplicate assignments of selectors Signed-off-by: PuBHARGAVI <[email protected]> * feat(INJI-436): add telemetry events to track the app generated warnings Signed-off-by: PuBHARGAVI <[email protected]> * fix(INJI-436): remove duplicate telemetry constant import in QR login file Signed-off-by: PuBHARGAVI <[email protected]> * refactor(INJI-436): remove unused app id import in app.tsx Signed-off-by: PuBHARGAVI <[email protected]> --------- Signed-off-by: PuBHARGAVI <[email protected]>
Telemetry events are added to tack App was reset and VC tampered scenarios