Feature/add govuk search #1117
Feature/add govuk search #1117
Conversation
|
Sorry for being so slow on this. The approach looks good just wondering if we have different ruff settings somehow? It looks like there's some reformatting happening? Are you running make format or doing something else? Not sure what's happening with the conflicts, probably just because we changed a lot while this branch was out? |
@jamesrichards4 Not really sure what is happening with the formatting. I did have some pending format changes before you reviewed, but even after using |
django_app/redbox_app/redbox_core/migrations/0059_merge_20241029_1135.py
Outdated
Show resolved
Hide resolved
| @@ -218,7 +216,7 @@ def save_ai_message( | |||
| url=citation_source.source, | |||
| text=citation_source.highlighted_text_in_source, | |||
| page_numbers=citation_source.page_numbers, | |||
| source=Citation.Origin(citation_source.source_type.title()), | |||
| source=Citation.Origin(citation_source.source_type), | |||
There was a problem hiding this comment.
Since this comes from an enum now, we don't need to do any .title() stuff
Context
We want to add functionality to search https://www.gov.uk/ into the agentic workflow
Changes proposed in this pull request
Guidance to review
Useful way to check it is by asking for travel advice to cuba
Relevant links
Search API documentation - https://docs.publishing.service.gov.uk/repos/search-api.html
Things to check