Skip to content

Comments

fix: remove some unnecessary calls to the API#821

Merged
rossiam merged 1 commit intoSmartThingsCommunity:mainfrom
rossiam:lazily-call-list
Feb 23, 2026
Merged

fix: remove some unnecessary calls to the API#821
rossiam merged 1 commit intoSmartThingsCommunity:mainfrom
rossiam:lazily-call-list

Conversation

@rossiam
Copy link
Collaborator

@rossiam rossiam commented Feb 23, 2026

Commands that used choose<Thing> functions were calling the list endpoints even if the id was already chosen. This makes that list lazy so it won't actually call the API until it's used.

@rossiam rossiam requested a review from a team February 23, 2026 19:53
@changeset-bot
Copy link

changeset-bot bot commented Feb 23, 2026

🦋 Changeset detected

Latest commit: 658f8cf

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@smartthings/cli Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@rossiam rossiam merged commit d71a676 into SmartThingsCommunity:main Feb 23, 2026
4 checks passed
@rossiam rossiam deleted the lazily-call-list branch February 23, 2026 20:22
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.

2 participants