-
Notifications
You must be signed in to change notification settings - Fork 26
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
Update index.md #114
base: main
Are you sure you want to change the base?
Update index.md #114
Conversation
Added a note on Start/End Session events to clarify that they do count towards the event volume.
🦙 MegaLinter status: ✅ SUCCESS
See detailed report in MegaLinter reports |
This pull request is automatically being deployed by Amplify Hosting (learn more). |
@@ -575,8 +575,9 @@ You can adjust the time window for which sessions are extended. The default sess | |||
Amplitude amplitude = new Amplitude(configuration); | |||
``` | |||
|
|||
By default, Amplitude automatically sends the '[Amplitude] Start Session' and '[Amplitude] End Session' events. Even though these events aren't sent, sessions are still tracked by using `session_id`. | |||
You can also disable those session events. | |||
By default, Amplitude sends the '[Amplitude] Start Session' and '[Amplitude] End Session' events. Even though these events aren't sent, sessions are still tracked by using `session_id`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@zheniasemenina This paragraph is unclear. First sentence says that Amplitude sends those events, but the second says otherwise. Can you confirm which is correct? Is it the same as the iOS-Swift article?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@markzegarelli This is incorrect, but it stems from the previous version of this SDK: it used to say 'By default, '[Amplitude] Start Session' and '[Amplitude] End Session' events aren't sent. Even though these events..'
So we can get rid of this piece now.
Added a note on Start/End Session events to clarify that they do count towards the event volume.
Amplitude Developer Docs PR
Description
Describe your changes.
Deadline
When do these changes need to be live on the site?
Change type
PR checklist:
mkdocs serve
.mkdocs serve
didn't generate any failures.@amplitude-dev-docs