Skip to content

Conversation

@sukantoraymond
Copy link
Contributor

Add user tracking to emulator


For contributor use:

  • Targeted PR against master branch
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work
  • Code follows the standards mentioned here
  • Updated relevant documentation
  • Re-reviewed Files changed in the Github PR explorer
  • Added appropriate labels

var state *flowkit.State
var err error
loader := &afero.Afero{Fs: afero.NewOsFs()}
command.HandleUserTracking(Cmd, loader)
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

need to add this since emulator is not part of addtoparent

@devbugging
Copy link
Contributor

Have you manually tested this usage of emulator and other commands?

@sukantoraymond
Copy link
Contributor Author

Already tested, they are being tracked in mixpanel

@devbugging devbugging self-requested a review August 23, 2022 16:42
@sukantoraymond sukantoraymond merged commit bd3404c into master Aug 23, 2022
@sukantoraymond sukantoraymond deleted the improvement/mixpanel-emulator branch August 23, 2022 16:50
@sukantoraymond sukantoraymond self-assigned this Aug 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants