search

go-telegram / bot

Telegram Bot API Go framework
MIT License
507 stars 46 forks source link

Should all models be updated to accept UserID as int64? #18

Closed iamelevich closed 1 year ago

iamelevich commented 1 year ago

In types we have UserID as int64 https://github.com/go-telegram/bot/blob/de923f1916d5a45abf121e41f86ec01e1d20ac82/models/user.go#L11 But in params we have it as int. For example in GetUserProfilePhotos https://github.com/go-telegram/bot/blob/de923f1916d5a45abf121e41f86ec01e1d20ac82/methods_params.go#L293

And it makes me really confusing about how to use it.

negasus commented 1 year ago

You right. I'll fix it soon

iamelevich commented 1 year ago

You right. I'll fix it soon

If you didn't started I can create PR today for it today.

negasus commented 1 year ago

Great. Waiting for PR

iamelevich commented 1 year ago

Great. Waiting for PR

Please check: https://github.com/go-telegram/bot/pull/19