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

feat(blocks): Add jina fact checker block #8409

Merged
merged 19 commits into from
Dec 2, 2024
Merged

feat(blocks): Add jina fact checker block #8409

merged 19 commits into from
Dec 2, 2024

Conversation

Bentlybro
Copy link
Member

@Bentlybro Bentlybro commented Oct 23, 2024 •
edited
Loading

Background

This adds the new Jina fact checker api as a block
https://jina.ai/news/fact-checking-with-new-grounding-api-in-jina-reader/

video to show it working with the new auth system!

2024-11-26.11-23-37.mp4

@Bentlybro Bentlybro requested a review from a team as a code owner October 23, 2024 12:46
@Bentlybro Bentlybro requested review from Torantulino and majdyz and removed request for a team October 23, 2024 12:46
@github-actions github-actions bot added the platform/frontend AutoGPT Platform - Front end label Oct 23, 2024
@github-actions github-actions bot added size/l and removed size/m labels Oct 23, 2024
ntindle
ntindle reviewed Oct 23, 2024
View reviewed changes
Copy link
Member

@ntindle ntindle left a comment

Choose a reason for hiding this comment

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

I've made some changes in #8403 that will likely impact this

validate that the API key works via .env as I wouldn't expect that to work with credits and what not based on current system design

autogpt_platform/backend/.env.example Outdated Show resolved Hide resolved
@github-actions github-actions bot added the conflicts Automatically applied to PRs with merge conflicts label Oct 24, 2024
@github-actions GitHub Actions
Copy link
Contributor

This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.

@github-actions github-actions bot removed platform/frontend AutoGPT Platform - Front end conflicts Automatically applied to PRs with merge conflicts labels Oct 24, 2024
@github-actions GitHub Actions
Copy link
Contributor

Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly.

@github-actions GitHub Actions
Copy link
Contributor

This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.

@github-actions github-actions bot added the conflicts Automatically applied to PRs with merge conflicts label Oct 31, 2024
@majdyz majdyz removed their request for review November 5, 2024 08:45
@Bentlybro
Copy link
Member Author

@ntindle ill get this sorted today

@github-actions github-actions bot removed the conflicts Automatically applied to PRs with merge conflicts label Nov 26, 2024
@github-actions GitHub Actions
Copy link
Contributor

Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly.

@github-actions github-actions bot added size/m and removed size/l labels Nov 26, 2024
@netlify Netlify
Copy link

netlify bot commented Nov 26, 2024 •
edited
Loading

✅ Deploy Preview for auto-gpt-docs canceled.

Name Link
🔨 Latest commit 4afc30b
🔍 Latest deploy log https://app.netlify.com/sites/auto-gpt-docs/deploys/674d8b7500214d0008f244cd

@Bentlybro Bentlybro requested a review from ntindle November 26, 2024 11:33
ntindle
ntindle requested changes Nov 27, 2024
View reviewed changes
Copy link
Member

@ntindle ntindle left a comment

Choose a reason for hiding this comment

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

Generate a new ID but after that it should be good

@Bentlybro Bentlybro requested a review from ntindle November 27, 2024 10:32
@netlify Netlify
Copy link

netlify bot commented Nov 27, 2024 •
edited
Loading

✅ Deploy Preview for auto-gpt-docs-dev canceled.

Name Link
🔨 Latest commit 4afc30b
🔍 Latest deploy log https://app.netlify.com/sites/auto-gpt-docs-dev/deploys/674d8b759202060008ab4c86

@aarushik93 aarushik93 enabled auto-merge November 29, 2024 13:40
@aarushik93 aarushik93 disabled auto-merge December 1, 2024 15:27
@aarushik93 aarushik93 enabled auto-merge December 1, 2024 15:27
@aarushik93 aarushik93 self-requested a review December 1, 2024 15:30
@Bentlybro Bentlybro disabled auto-merge December 2, 2024 10:32
@Bentlybro Bentlybro merged commit be7f912 into dev Dec 2, 2024
19 checks passed
@Bentlybro Bentlybro deleted the add-jina-fact-checker-block branch December 2, 2024 10:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aarushik93 aarushik93 aarushik93 approved these changes

@Torantulino Torantulino Awaiting requested review from Torantulino Torantulino is a code owner automatically assigned from Significant-Gravitas/maintainers

@ntindle ntindle Awaiting requested review from ntindle

Assignees
No one assigned
Labels
platform/backend AutoGPT Platform - Back end platform/blocks size/m
Projects
AutoGPT development kanban
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Bentlybro @ntindle @aarushik93