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.

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

fix: setOnce operation name #116

Closed
wants to merge 1 commit into from

Conversation

dams229
Copy link

@dams229 dams229 commented Feb 12, 2024

Summary

  • Fix setOnce operation name in code so it is handled correctly by Amplitude

Checklist

  • Does your PR title have the correct title format?
  • Does your PR have a breaking change?: No

@dams229 dams229 changed the title fix: setOnce operation name (#115) fix: setOnce operation name Feb 12, 2024
@qingzhuozhen
Copy link
Contributor

Hi @dams229 thanks for submitting the fix. There are couple unit test failing so I clone the change and fix the unit tests in another pull request. Will include the fix in next release.

@dams229 dams229 closed this Feb 14, 2024
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.

2 participants