Replace isAnonymous property with externalUserId #1

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

Closed

denis-rossati wants to merge 1 commit into master from update-is-anonymous

Member

denis-rossati commented Nov 19, 2024

Summary

This PR will remove the isAnonymous property and introduce the externalUserId property to follow the Session changes

Checklist

My code follows the style guidelines of this project
I have performed a self-review of my own code
I have commented my code, particularly in hard-to-understand areas
I have made corresponding changes to the documentation
My changes generate no new warnings
I have added tests that prove my fix is effective or that my feature works
New and existing unit tests pass locally with my changes
Any dependent changes have been merged and published in downstream modules
I have checked my code and corrected any misspellings


          Remove isAnonymous property

38bf5f4

denis-rossati added the bc-break label

denis-rossati requested a review from marcospassos

November 19, 2024 12:42

denis-rossati self-assigned this

Member

Fryuni commented Nov 21, 2024

This repo is generated automatically. We shouldn't have direct PRs to it.

Fryuni closed this

Fryuni deleted the update-is-anonymous branch

November 21, 2024 12:22

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels