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.

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

community: Made some Jira fields optional for agent to work correctly #29876

Merged
merged 1 commit into from
Feb 20, 2025

Conversation

dokato
Copy link
Contributor

@dokato dokato commented Feb 19, 2025

Description: Two small changes have been proposed here:
(1)
Previous code assumes that every issue has a priority field. If an issue lacks this field, the code will raise a KeyError.
Now, the code checks if priority exists before accessing it. If priority is missing, it assigns None instead of crashing. This prevents runtime errors when processing issues without a priority.

(2)

Also If the "style" field is missing, the code throws a KeyError. .get("style", None) safely retrieves the value if present.

Issue: #29875

Dependencies: N/A

@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Feb 19, 2025
Copy link

vercel bot commented Feb 19, 2025

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

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
langchain ⬜️ Ignored (Inspect) Visit Preview Feb 19, 2025 11:25am

@dosubot dosubot bot added community Related to langchain-community 🤖:bug Related to a bug, vulnerability, unexpected error with an existing feature labels Feb 19, 2025
@dosubot dosubot bot added the lgtm PR looks good. Use to confirm that a PR is ready for merging. label Feb 20, 2025
@ccurme ccurme merged commit 92b415a into langchain-ai:master Feb 20, 2025
19 checks passed
@dokato dokato deleted the fix-jira-tool branch February 20, 2025 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤖:bug Related to a bug, vulnerability, unexpected error with an existing feature community Related to langchain-community lgtm PR looks good. Use to confirm that a PR is ready for merging. size:S This PR changes 10-29 lines, ignoring generated files.
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants