#1048: Update laravel package code to reflect namespace changes

php-telegram-bot / laravel

Laravel package for PHP Telegram Bot Library

Other

168 stars 50 forks source link

Closed debo closed 1 year ago

debo commented 4 years ago

PR to support the suggestion made here of changing the core library namespace.

TiiFuchs commented 1 year ago

Closed due to new code base with v2.0

debo commented 1 year ago

You could have asked me to update the PR to be compliant rather than close it to be fair, but hey. Is it worth for me to open a new one?

TiiFuchs commented 1 year ago

We completely replaced the code base of the laravel integration. Therefore this PR doesn't fit anymore.

I didn't check exactly what it was about, since the codebase is different and it's not applicable anyway.

Sorry for the inconvenience.

debo commented 1 year ago

It was about unifying the namespaces and make them more meaningful and in context. You can see the details in the linked issue.

Happy to open a new PR compatible with the current codebase. Just let me know also because it Hacktoberfest now ;)