Skip to content

fix(recap): Update associate_related_instances check for better logs#7212

Merged
albertisfu merged 3 commits intomainfrom
morgan/related-instances-log-tweak
Apr 10, 2026
Merged

fix(recap): Update associate_related_instances check for better logs#7212
albertisfu merged 3 commits intomainfrom
morgan/related-instances-log-tweak

Conversation

@MorganBennetDev
Copy link
Copy Markdown
Contributor

Summary

Updates check for bad rd_id parameter in associate_related_instances to stop logging erroneous errors.

Deployment

This PR should:

  • skip-deploy (skips everything below)
    • skip-web-deploy
    • skip-celery-deploy
    • skip-cronjob-deploy
    • skip-daemon-deploy

Copy link
Copy Markdown
Contributor

@albertisfu albertisfu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @MorganBennetDev just a small suggestion.

Co-authored-by: Alberto Islas <albertisfu@gmail.com>
Copy link
Copy Markdown
Contributor

@albertisfu albertisfu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, looks good. Merging.

@albertisfu albertisfu enabled auto-merge April 10, 2026 18:16
@albertisfu albertisfu merged commit 93e2481 into main Apr 10, 2026
9 checks passed
@albertisfu albertisfu deleted the morgan/related-instances-log-tweak branch April 10, 2026 18:26
@github-project-automation github-project-automation bot moved this from To Do to Done in Sprint (Web Team) Apr 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants