Feat: can custom full url on a request

[ah-rahimi]

More implementation on #743 to customize the URL for a request.

about tests: It works like the setBaseUrl() and setHttpProxy() methods.

[ah-rahimi]

Hi @fabio-ivona, @MarioGattolla.
Hope you are well.

Please check this for merge.
Thanks

[fabio-ivona]

@ah-rahimi isn't enough the customizable bot url method to achieve this?

[ah-rahimi]

@fabio-ivona, since I'm manually using the Telegraph Facade, that's not useful for me.

[ah-rahimi]

Hi, when will you merge it?
@fabio-ivona

[ah-rahimi]

We need this for tomorrow.
We have a release
Thanks 🙏

@fabio-ivona

[fabio-ivona]

Sorry @ah-rahimi I'm not 100% convinced of this, too much cognitive overload on the package

I need time to dive deeper into it and think a cleaner solution

[ah-rahimi]

Yes, you're right
That's no problem.

[ah-rahimi]

Then, as you implemented, you have to define a custom URL for all BOTs
and this may not be interesting.
but this way you only define a custom URL for what you want.

And we need to create a new model and extend from TelegraphBot so we can customize it.

[fabio-ivona]

Then, as you implemented, you have to define a custom URL for all BOTs and this may not be interesting. but this way you only define a custom URL for what you want.

And we need to create a new model and extend from TelegraphBot so we can customize it.

That's usually how customization works for Laravel

I'm planning a setup system in service provider for the upcoming V2 that will be a complete rewrite

Btw, in this weekend I'll work on a better url customization system for the V1

[ah-rahimi]

Hi @fabio-ivona ,
Do you have any plans for this?
We still need this feature.
We will be glad to merge it sooner.

[fabio-ivona]

Hi @fabio-ivona , Do you have any plans for this? We still need this feature. We will be glad to merge it sooner.

I'll work on this during this week, we are under an heavy workload, so very few time available

[ah-rahimi]

Thanks @fabio-ivona