Skip to content

Conversation

@mmrj
Copy link
Contributor

@mmrj mmrj commented May 13, 2025

Requirements

  • I have added test coverage for new or changed functionality
  • I have followed the repository's pull request submission guidelines
  • I have validated my changes against all supported platform versions

Related issues

Found during https://launchdarkly.atlassian.net/browse/DOCS-2295, https://github.com/launchdarkly/ld-docs-private/pull/5935

Describe the solution you've provided

The beforeIdentify and afterIdentify methods in the Hook interface had identical documentation about when they're called, which is not how they actually behave.

@mmrj mmrj requested a review from a team as a code owner May 13, 2025 17:53
@github-actions
Copy link
Contributor

@launchdarkly/js-sdk-common size report
This is the brotli compressed size of the ESM build.
Size: 23840 bytes
Size limit: 25000

@github-actions
Copy link
Contributor

@launchdarkly/js-client-sdk size report
This is the brotli compressed size of the ESM build.
Size: 20702 bytes
Size limit: 21000

@github-actions
Copy link
Contributor

@launchdarkly/js-client-sdk-common size report
This is the brotli compressed size of the ESM build.
Size: 15786 bytes
Size limit: 20000

@kinyoklion kinyoklion changed the title docs: afterIdentify occurs at a different time than beforeIdentify chore(docs): afterIdentify occurs at a different time than beforeIdentify May 13, 2025
@kinyoklion kinyoklion merged commit c985daa into main May 13, 2025
24 checks passed
@kinyoklion kinyoklion deleted the mmrj-DOCS-2295-additional-js-hooks-support branch May 13, 2025 20:23
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