Configuration mmpy-bot with mattermost about mmpy_bot

[attzonko]

There was a recent fix to the mattermostdriver package related to websocket connection. Could you try with the latest mattermostdriver from the HEAD of that repo?

[unode]

As we are now using a different driver, could you please re-test if the latest version from the main branch still displays this bug and report here?
Thanks!

[cristianjuve]

We're receiving the same issue after upgrade to mattermost 7.1.2 from 5.38.2.
Error with mmpy-bot=2.0.2

If we enable debug mode then it works.

Error with mmpy-bot=2.1.0

In this case if we enable debug mode it doesn't work.

We're using a custom plugin and we don't understand what's the function that is failing on our code. I share with you part of our code to try to find where is the issue.

Thanks!

[unode]

In the first case, there's a problem in mattermostdriver that was since fixed but an upgrade is required.
Regarding the plugin.initialize() error in 2.1.0, there was an API change that introduced another argument (plugin_manager).

See the difference in the function signature between 2.0.2

mmpy_bot/mmpy_bot/plugins/base.py

Line 40 in 81e29ea

def initialize(self, driver: Driver, settings: Optional[Settings] = None):

and 2.1.0

mmpy_bot/mmpy_bot/plugins/base.py

Line 34 in 641b918

def initialize(

[cristianjuve]

How can I fix the issue with mattermostdriver? upgrading mattermost(version?)?? or mmpy-bot?
Regarding the API changes, how should I initialize the Bot? I'm not sure how to pass the plugins arg Optional[Union[List[Plugin], PluginManager]]

[unode]

You can continue using:

plugins=[SealPlugin()]

but you need to update the signature of def initialize(driver, plugin_manager, settings): in your plugin.

You don't need to use your own PluginManager instance unless you want to customize/override its behavior.

[unode]

Please see #306 for a follow up on the mattermostdriver issue.

I believe the issue is solved by embl-bio-it/python-mattermost-autodriver@b699cb9 but a version of mmpy_bot that uses this fix hasn't been released yet.
Please try the latest code from the main branch and lets us know if the problem is gone.

I'm using it on Mattermost 7.1.2 without problems.

[cristianjuve]

it works!! thnks! :)

[unode]

Closing as resolved in main.

[attzonko]

Just made release 2.1.1 from the latest main

Thanks for confirming the fixes work for you @cristianjuve