Skip to content

feat: added typing for keys and default value for VariableValue decorator - #1109

Merged
jsalaber merged 2 commits into
mainfrom
nestjs-variable-value-typing
Jul 7, 2025
Merged

feat: added typing for keys and default value for VariableValue decorator#1109
jsalaber merged 2 commits into
mainfrom
nestjs-variable-value-typing

Conversation

@jsalaber

@jsalaber jsalaber commented Jul 4, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

Changes

  • changed typing to make passed in key and default value abide by variable types generated by the CLI

@vercel

vercel Bot commented Jul 4, 2025 •
edited
Loading

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
js-sdks-web-elements ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 4, 2025 7:10pm
js-sdks-with-provider ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 4, 2025 7:10pm
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
js-sdks-next-js-page-router ⬜️ Ignored (Inspect) Jul 4, 2025 7:10pm

@jsalaber
jsalaber requested a review from a team July 4, 2025 19:11
@jsalaber
jsalaber merged commit c1578e6 into main Jul 7, 2025
@jsalaber
jsalaber deleted the nestjs-variable-value-typing branch July 7, 2025 13:28
jonathannorris added a commit that referenced this pull request May 26, 2026
@jonathannorris
chore: resolve uuid and qs dependabot security alerts
3e57c18 
- uuid ^8.3.2 -> ^11.1.1 in package.json and sdk/js/package.json (runtime, alerts #1113/#1114)
- uuid resolution ^11.1.1 added to root and all e2e sub-project package.json files (alerts #1105-#1109)
- qs resolutions added for exact-pinned 6.13.0 and 6.14.0 -> ^6.15.2 in root (alerts #1110-#1112)
@jonathannorris jonathannorris mentioned this pull request May 27, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jonathannorris jonathannorris jonathannorris approved these changes
@suthar26 suthar26 suthar26 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@jsalaber @jonathannorris @suthar26