Skip to content

Conversation

@m1ome
Copy link
Contributor

@m1ome m1ome commented Nov 19, 2019

Resolves #22 also it handles backwards compatibility with previous version.
That means you can use Nadia.get_me along with Nadia.get_me(:my_bot) and all will be totally fine.

Also bc changes not broken for:
Nadia.Config, Nadia.API, Nadia.Graph, Nadia.Graph.API

It shouldn't broke anything!

@m1ome
Copy link
Contributor Author

m1ome commented Nov 19, 2019

Please don't merge it, just wanna know your thoughts about it. I will update Nadia.Behaviour if all looks good to you!

@m1ome
Copy link
Contributor Author

m1ome commented Dec 23, 2019

@zhyu any news on this?

@zhyu
Copy link
Owner

zhyu commented Dec 26, 2019

@m1ome I'm so sorry for keeping you waiting :(

I really love the way you implementing it! Please update Nadia.Behaviour and add related tests if you have time, then I'll merge it!

Thanks

@m1ome
Copy link
Contributor Author

m1ome commented Dec 26, 2019

Try to do it ASAP

@skarpesh
Copy link

@m1ome are you going to work on it? I could continue your work if you don't have enough time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support multiple bots

3 participants