Skip to content

Conversation

@jgrantr
Copy link
Contributor

@jgrantr jgrantr commented Nov 18, 2025

  • the getSetting function actually returns a wrapped value and not a direct T as it was originally typed.

Note

Corrects settings API types to return a wrapped LeoSettingEntry<T> (with id and value) and bumps version to 7.1.16.

  • Types (DynamoDB):
    • Add LeoSettingEntry<T> with id and value.
    • Update settings API signatures to return wrapped entries:
      • getSettingPromise<T> now returns Promise<LeoSettingEntry<T>>.
      • getSetting<T> callback now yields LeoSettingEntry<T>.
  • Version:
    • Bump package version to 7.1.16.

Written by Cursor Bugbot for commit 72fec5c. This will update automatically on new commits. Configure here.

- the getSetting function actually returns a wrapped value and not a direct T as it was originally typed.
@ch-snyk-sa
Copy link

ch-snyk-sa commented Nov 18, 2025

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@jgrantr jgrantr merged commit 220c307 into master Nov 18, 2025
8 checks passed
@jgrantr jgrantr deleted the feature/ES-2352-fix-getSetting-typing branch November 18, 2025 05:41
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