-
Notifications
You must be signed in to change notification settings - Fork 30
Comparing changes
Open a pull request
base repository: launchdarkly/js-core
base: main
head repository: launchdarkly/js-core
compare: ta/fdv2-temporary-holding
- 10 commits
- 67 files changed
- 2 contributors
Commits on Jan 22, 2025
-
chore: preparation work for FDv2 feature store and persistence support (
#725) Splitting these changes up into multiple PRs to keep them manageable and because we can't merge directly to main as `applyChanges` is dependent on transaction support in the PersistentDataStore. Existing tests are exercising applyChanges, but through init, upsert, and delete. Plan to add applyChanges tests in ta/fdv2-temporary-holding after transaction support is added to PersistentDataStore in a subsequent PR as part of SDK-1029
Configuration menu - View commit details
-
Copy full SHA for 179b199 - Browse repository at this point
Copy the full SHA 179b199View commit details
Commits on Jan 27, 2025
-
feat: adds fdv2 transactional persistent store (#747)
**Related issues** SDK-1029 **Describe the solution you've provided** Adds TransactionalPersistentStore containing memory store and persistence store. Swaps on basis being provided as necessary. Planning to add tests on holding branch (PR target branch).
Configuration menu - View commit details
-
Copy full SHA for c2debf7 - Browse repository at this point
Copy the full SHA c2debf7View commit details
Commits on Feb 3, 2025
-
Configuration menu - View commit details
-
Copy full SHA for 5dbe832 - Browse repository at this point
Copy the full SHA 5dbe832View commit details
Commits on Mar 28, 2025
-
chore: adds CompositeDataSource for FDv2 support (#787)
**Requirements** - [x] I have added test coverage for new or changed functionality - [x] I have followed the repository's [pull request submission guidelines](../blob/main/CONTRIBUTING.md#submitting-pull-requests) - [ ] I have validated my changes against all supported platform versions Will be done on target temp branch eventually. **Related issues** SDK-857
Configuration menu - View commit details
-
Copy full SHA for d1d9e0f - Browse repository at this point
Copy the full SHA d1d9e0fView commit details
Commits on Apr 7, 2025
-
chore: Adds LDDataSystemOptions for configuring the Data System. (#794)
**Requirements** - [x] I have added test coverage for new or changed functionality - [x] I have followed the repository's [pull request submission guidelines](../blob/main/CONTRIBUTING.md#submitting-pull-requests) - [ ] I have validated my changes against all supported platform versions Will be done on target temp branch eventually. **Related issues** SDK-857 and SDK-1073 --------- Co-authored-by: Todd Anderson <tanderson@Todds-MacBook-Pro.local>
Configuration menu - View commit details
-
Copy full SHA for e2c26af - Browse repository at this point
Copy the full SHA e2c26afView commit details
Commits on Apr 11, 2025
-
feat: adds polling synchronizer support (#816)
**Requirements** - [x] I have added test coverage for new or changed functionality - [x] I have followed the repository's [pull request submission guidelines](../blob/main/CONTRIBUTING.md#submitting-pull-requests) - [ ] I have validated my changes against all supported platform versions This will be done at a later time on the target branch during integration testing. **Related issues** SDK-858 and SDK-851 **Describe the solution you've provided** Refactors payload processing to be reusable between polling data sources and streaming data sources. Adds OneShotInitializers and PollingSynchronizer. Adds support to contract tests for sdk-test-harness/feat/fdv2 branch. --------- Co-authored-by: Todd Anderson <tanderson@Todds-MacBook-Pro.local>
Configuration menu - View commit details
-
Copy full SHA for c1089a5 - Browse repository at this point
Copy the full SHA c1089a5View commit details
Commits on Apr 22, 2025
-
feat: CompositeDataSource blacklists DataSources that report unrecove…
…rable error (#826) **Requirements** - [x] I have added test coverage for new or changed functionality - [x] I have followed the repository's [pull request submission guidelines](../blob/main/CONTRIBUTING.md#submitting-pull-requests) - [ ] I have validated my changes against all supported platform versions To be done on target branch. --------- Co-authored-by: Todd Anderson <tanderson@Todds-MacBook-Pro.local>
Configuration menu - View commit details
-
Copy full SHA for aee6479 - Browse repository at this point
Copy the full SHA aee6479View commit details
Commits on Apr 29, 2025
-
feat: falls back to FDv1 polling if flag delivery sends fallback head…
…er (#832) **Requirements** - [x] I have added test coverage for new or changed functionality - [x] I have followed the repository's [pull request submission guidelines](../blob/main/CONTRIBUTING.md#submitting-pull-requests) - [ ] I have validated my changes against all supported platform versions Will validate on holding branch later. **Related issues** SDK-1179 **Describe the solution you've provided** Adds FDv1 fallback error type. FDv2 datasources now report that error type if they see the fallback header from the FDv2 endpoints. Updates FDv2 polling datasource to support being constructed in a FDv1 to FDv2 conversion mode. In that mode, the response from an FDv1 endpoint will be converted to an FDv2 payload and passed through the FDv2 payloadProcessor. CompositeDataSource now supports an FDv1 fallback synchronizer list. If any datasource reports the FDv1 fallback error, the CompositeDataSource switches to use the fallback synchronizer list. This PR also contains tweaks to get the holding branch working with the latest contract tests that got some tweaks during recent integration testing. --------- Co-authored-by: Todd Anderson <tanderson@Todds-MacBook-Pro.local>
Configuration menu - View commit details
-
Copy full SHA for 063c778 - Browse repository at this point
Copy the full SHA 063c778View commit details
Commits on May 9, 2025
-
feat: now utilizes FDv2 basis param for more efficient polling (#843)
**Requirements** - [x] I have added test coverage for new or changed functionality - [x] I have followed the repository's [pull request submission guidelines](../blob/main/CONTRIBUTING.md#submitting-pull-requests) - [ ] I have validated my changes against all supported platform versions Will be done on target branch in the future as part of integration testing. **Related issues** SDK-1044 **Describe the solution you've provided** Pipes selector through datasource and feature store layers / APIs. InMemoryFeatureStore holds the selector in memory at runtime. When datasources need the selector for making their requests, they invoke a selectorGetter lambda that is hooked up to the InMemoryFeatureStore. --------- Co-authored-by: Todd Anderson <tanderson@Todds-MacBook-Pro.local>
Configuration menu - View commit details
-
Copy full SHA for 82f9ff1 - Browse repository at this point
Copy the full SHA 82f9ff1View commit details
Commits on May 12, 2025
-
feat: now utilizes FDv2 basis param for streaming (#847)
**Requirements** - [x] I have added test coverage for new or changed functionality - [x] I have followed the repository's [pull request submission guidelines](../blob/main/CONTRIBUTING.md#submitting-pull-requests) - [x] I have validated my changes against all supported platform versions **Related issues** SDK-1220 Co-authored-by: Todd Anderson <tanderson@Todds-MacBook-Pro.local>
Configuration menu - View commit details
-
Copy full SHA for b1d1bc6 - Browse repository at this point
Copy the full SHA b1d1bc6View commit details
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff main...ta/fdv2-temporary-holding