Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix(service): add missing nil check in includeIteratorComponentDetail() #831

Merged
merged 1 commit into from
Nov 11, 2024
Merged

fix(service): add missing nil check in includeIteratorComponentDetail() #831

merged 1 commit into from
Nov 11, 2024
+6 −4

Conversation

donch1989
Copy link
Member

Because

  • A nil check is missing in includeIteratorComponentDetail()

This commit

  • Fixes the bug by adding the necessary nil check.

@donch1989 donch1989 changed the title fix: add missing nil check in includeIteratorComponentDetail() fix(service): add missing nil check in includeIteratorComponentDetail() Nov 11, 2024
@donch1989 donch1989 merged commit 9cb5e9e into main Nov 11, 2024
14 of 15 checks passed
@donch1989 donch1989 deleted the hotfix/v0.46 branch November 11, 2024 15:35
@droplet-bot droplet-bot mentioned this pull request Nov 11, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jvallesm jvallesm Awaiting requested review from jvallesm jvallesm is a code owner

@chuang8511 chuang8511 Awaiting requested review from chuang8511 chuang8511 is a code owner

Assignees
No one assigned
Labels
instill vdp
Projects
No open projects
🔮 Instill Core
Status: No status
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@donch1989 @droplet-bot