search

mattermost / mattermost-plugin-ai

Mattermost Copilot plugin supporting multiple LLMs
https://mattermost.com/copilot
Apache License 2.0
137 stars 31 forks source link

[feat] updating the rudder key using the plugins key environment variable #222

Closed phoinixgrr closed 4 months ago

phoinixgrr commented 4 months ago

Reverting https://github.com/mattermost/mattermost-plugin-ai/pull/221 To be merged after: https://github.com/mattermost/delivery-platform/pull/156

Ticket: https://mattermost.atlassian.net/browse/CLD-8054

phoinixgrr commented 4 months ago

nit: shouldn't we be using copilot rather than plugin-ai for naming?

@ifoukarakis Refers to mattermost-plugin-ai plugin , should be easier to be correlated with the actual usage. We can change it if you think that is needed though.

Let me know.

ifoukarakis commented 4 months ago

nit: shouldn't we be using copilot rather than plugin-ai for naming?

@ifoukarakis Refers to mattermost-plugin-ai plugin , should be easier to be correlated with the actual usage. We can change it if you think that is needed though.

Let me know.

I'm ok with either. mattermost-plugin-ai is the name of the repo, but README.md mentions Mattermost Copilot Plugin, so I was thinking if there's an agreed naming so that we're consistent. Again, that's nitpicking, so not a blocker.