Skip to content
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.

Sign up for GitHub

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

Jump to bottom

feat(mixpanel): Update/add functions #2697

Merged
merged 2 commits into from
Sep 15, 2018
Merged

feat(mixpanel): Update/add functions #2697

merged 2 commits into from
Sep 15, 2018

Conversation

SpellChucker
Copy link
Contributor

@SpellChucker SpellChucker commented Sep 4, 2018 •
edited
Loading

  • alias does not require an originalId.
  • identify accepts a usePeople parameter for the iOS SDK.
  • New methods:
    append
    deleteUser
    trackCharge
    unset
    union

@SpellChucker
Copy link
Contributor Author

Not sure why tests are failing, doesn't seem to be related to changes.

@paulstelzer
Copy link
Contributor

@SpellChucker No, it's not you - had the same problems because their test not install cordova-plugin-my-plugin correctly ;) - so just ignore it

@danielsogl danielsogl self-assigned this Sep 9, 2018
@danielsogl danielsogl changed the title fix(mixpanel): Update/add functions for mixpanel feat(mixpanel): Update/add functions Sep 9, 2018
@danielsogl danielsogl added the target: patch This PR is targeted for the next patch release label Sep 9, 2018
@danielsogl danielsogl merged commit ac467c5 into danielsogl:master Sep 15, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@danielsogl danielsogl danielsogl approved these changes

Assignees

@danielsogl danielsogl

Labels
target: patch This PR is targeted for the next patch release
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@SpellChucker @paulstelzer @danielsogl