Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@Sarthak5598
Copy link
Member

No description provided.

Copy link
Member Author

@Sarthak5598 Sarthak5598 left a comment

Choose a reason for hiding this comment

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

Not ready for merge now , Just want to keep it open as everyone is working on it , Would like to know everyones opinion

@Sarthak5598 Sarthak5598 changed the title added langchain adn openai to poetry Write a function in views.py to integrate the model with the web tool. #2194 Jun 20, 2024
@Sarthak5598
Copy link
Member Author

Hey, can anyone help me with this? For some reason, the AutoLabel function isn't getting the bugdescription. I've console logged everything, and it's correct from the report.html side. It's even getting the label from the function (JSON response), but it can't get bugdescription from report.html.

@Sarthak5598
Copy link
Member Author

Works now !!, no need for a review as of now, thanks

@Sarthak5598
Copy link
Member Author

The new implementation works as intended for setting the label but the issue is that the bug reports are no longer being posted on the main page as they used to be. It seems that the issue may be related to how the bug reports are handled after the label selection is done. please help

@Sarthak5598 Sarthak5598 enabled auto-merge June 29, 2024 04:51
@Sarthak5598 Sarthak5598 requested a review from arkid15r June 29, 2024 04:51
Copy link

@arkid15r arkid15r left a comment

Choose a reason for hiding this comment

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

LGTM

@Sarthak5598 Sarthak5598 merged commit 97a46ba into OWASP-BLT:main Jun 29, 2024
@sentry
Copy link

sentry bot commented Jun 30, 2024

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ OpenAIError: The api_key client option must be set either by passing api_key to the client or by setting the O... /issue-auto-label/ View Issue

Did you find this useful? React with a 👍 or 👎

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