Skip to content

Conversation

@MinnDevelopment
Copy link
Contributor

The $ prefix is optional and dropped server-side.

Copy link
Contributor

@vvito7 vvito7 left a comment

Choose a reason for hiding this comment

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

Should these ones also be edited?

"properties": {
"$os": "linux",
"$browser": "my_library",
"$device": "my_library"

@MinnDevelopment
Copy link
Contributor Author

MinnDevelopment commented Jun 14, 2022

Updated, but maybe that other example should just link to the event docs? Looks like it already does.

@DV8FromTheWorld
Copy link
Member

Didn't realize this PR was open when I opened #5067

Going to close this one in favor of 5067 as we're going to write a changelog entry in that one too.

@MinnDevelopment MinnDevelopment deleted the identify-payload branch June 17, 2022 16:54
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