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

refactor: update SvelteLDClient to use compat SDK and improve initial… #4

Merged
merged 1 commit into from
Dec 3, 2024

Conversation

nosnibor89
Copy link
Owner

This pull request includes several changes to the SvelteLDClient and its associated tests and dependencies. The key changes involve updating the initialization of the LaunchDarkly client to include a user context, modifying the imports to use the compatibility layer, and updating dependencies in the package.json file.

Changes to the SvelteLDClient and Tests:

  • Updated the initialization of the LaunchDarkly client to include a user context. This change affects multiple test cases to ensure the context is passed correctly. (packages/sdk/svelte/__tests__/lib/client/SvelteLDClient.test.ts) [1] [2] [3] [4] [5] [6] [7]

  • Modified imports to use the compatibility layer of the LaunchDarkly SDK. This change ensures compatibility with the current version of the SDK. (packages/sdk/svelte/__tests__/lib/client/SvelteLDClient.test.ts, packages/sdk/svelte/src/lib/client/SvelteLDClient.ts) [1] [2]

Dependency Updates:

  • Updated various dependencies in the package.json file to their latest versions, including svelte, vite, and vitest. These updates ensure the project uses the latest features and security patches. (packages/sdk/svelte/package.json) [1] [2] [3]

Code Enhancements:

  • Refactored the initialize function in SvelteLDClient.ts to accept and properly handle the user context. This change ensures that the LaunchDarkly client is initialized with the correct context and improves the handling of flag values. (packages/sdk/svelte/src/lib/client/SvelteLDClient.ts) [1] [2]

@nosnibor89 nosnibor89 self-assigned this Dec 3, 2024
@nosnibor89 nosnibor89 merged commit 2365346 into feat/svelte-sdk Dec 3, 2024
40 of 43 checks passed
Copy link

github-actions bot commented Dec 3, 2024

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

Copy link

github-actions bot commented Dec 3, 2024

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

Copy link

github-actions bot commented Dec 3, 2024

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

@nosnibor89 nosnibor89 deleted the feat/svelte-sdk-client-improvements branch December 11, 2024 00:14
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.

1 participant