Skip to content

Conversation

@nosnibor89
Copy link
Contributor

@nosnibor89 nosnibor89 commented Dec 19, 2024

Requirements

  • I have added test coverage for new or changed functionality (no behavior change is mostly example project and docs)
  • I have followed the repository's pull request submission guidelines
  • I have validated my changes against all supported platform versions

Related issues

No issue.

Describe the solution you've provided

This pull request introduces a new Svelte example project to demonstrate the usage of @launchdarkly/svelte-client-sdk. The README of the example project includes steps to run such application that internally uses the SDK to interact with a boolean flag.

Also, this PR adds documentation for @launchdarkly/svelte-client-sdk itself with a "Getting Started" session along with more advanced use of the SDK's api.

Describe alternatives you've considered

I don't know what to write here.

Additional context

This is a follow up PR for #632. Where Svelte SDK was introduced.

After this, follow up PR for technical debt(upgrade Svelte 5 and improving test coverage ) should be expected

robinson-md and others added 21 commits October 23, 2024 17:04
intial refactor to use new "@launchdarkly/js-client-sdk"
[refactor] Update SvelteLDClient to use proxy for flag variations. Allows to track flag evaluations
…ents

refactor: update SvelteLDClient to use compat SDK and improve initial…
@nosnibor89 nosnibor89 changed the title Feat/svelte sdk example feat: add svelte example project and improving SDK docs Jan 6, 2025
@nosnibor89 nosnibor89 marked this pull request as ready for review January 6, 2025 21:04
@nosnibor89 nosnibor89 requested a review from a team as a code owner January 6, 2025 21:04
@abarker-launchdarkly abarker-launchdarkly changed the base branch from main to feat/svelte-sdk-example May 14, 2025 13:23
Copy link
Contributor

@abarker-launchdarkly abarker-launchdarkly left a comment

Choose a reason for hiding this comment

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

@nosnibor89 Thank you for the contribution. This all looks good but I'm first merging to a temporary holding branch to make a few editorial changes to the README to make it more consistent with our other documentation.

@abarker-launchdarkly abarker-launchdarkly merged commit 0b51885 into launchdarkly:feat/svelte-sdk-example May 14, 2025
2 checks passed
abarker-launchdarkly added a commit that referenced this pull request May 15, 2025
The contributed work was already reviewed in the original PR
(#722) which was merged into
this temporary holding branch to:
- Standardize the README files to be consistent with our other
documentation
- Add the build/test github action workflow for the Svelte SDK

Only the changes made in commit
79b42ae
need to be reviewed.

---------

Co-authored-by: Robinson Marquez <robinsonmadrid@outlook.com>
Co-authored-by: Robinson Marquez <nosnibor1989@gmail.com>
Co-authored-by: Robinson Marquez <robinsonmarquez@robinsonmarquezm4.local>
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