Integrate with the new wpai_has_ai_credentials filter#43
Merged
Conversation
…lama or return an error if something went wrong, like no connection. Add a new is_connected method that uses the get_models method to determine if the AI plugin should show things as being connected or not. This is needed since Ollama doesn't require an API key and thus the AI plugin's detection doesn't work
jeffpaul
approved these changes
Apr 13, 2026
Collaborator
jeffpaul
left a comment
jeffpaul
left a comment
There was a problem hiding this comment.
Tested and connector continues to work as expected
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description of the Change
The AI plugin doesn't see the Ollama Connector as being connected since it doesn't require an API key. A new
wpai_has_ai_credentialsfilter was introduced in that plugin that allows individual Connectors to override this behavior.This PR hooks into that filter and if we have models available, we assume Ollama is connected and return true.
How to test the Change
Changelog Entry
Credits
Props @dkotter
Checklist: