Skip to content

[Platform][Ollama] Remove prefix for ScopingHttpClient#1888

Merged
chr-hertel merged 1 commit into
symfony:mainfrom
Guikingone:fix/ollama_model_catalog
Apr 8, 2026
Merged

[Platform][Ollama] Remove prefix for ScopingHttpClient#1888
chr-hertel merged 1 commit into
symfony:mainfrom
Guikingone:fix/ollama_model_catalog

Conversation

@Guikingone

Copy link
Copy Markdown
Contributor
Q A
Bug fix? yes
New feature? no
Docs? yes
Issues Discussed in #1762 (comment).
License MIT

@Guikingone Guikingone changed the title refactor(ollama): remove prefix for ScopingHttpClient [Platform][Ollama] Remove prefix for ScopingHttpClient Apr 7, 2026
@Guikingone Guikingone marked this pull request as ready for review April 7, 2026 13:38
@carsonbot carsonbot added Bug Something isn't working Platform Issues & PRs about the AI Platform component Status: Needs Review labels Apr 7, 2026
@Guikingone

Copy link
Copy Markdown
Contributor Author

The CI is failing as I renamed the PR after creating it, no issue on tests 😅

@chr-hertel chr-hertel left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks, in general i feel we need to revisit how robust our / handling is - but that's maybe an easy prompt :)

@chr-hertel

Copy link
Copy Markdown
Member

Thank you @Guikingone.

@chr-hertel chr-hertel merged commit 22811f3 into symfony:main Apr 8, 2026
411 of 412 checks passed
@Guikingone Guikingone deleted the fix/ollama_model_catalog branch April 8, 2026 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Something isn't working Platform Issues & PRs about the AI Platform component Status: Reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants