-
Notifications
You must be signed in to change notification settings - Fork 100
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Улучшение класса Keyboard #595
Conversation
Для чего нужен флаг |
Нет, ну можно было бы конечно добавлять пустой список в конец, и чтобы |
Я считаю, что это должен решать юзер, а не фреймворк. |
Опять таки, в каком таком сценарии юзер захочет передать в API пустую строку и получить невнятную ошибку в ответ, которая больше заблуждает, чем объясняет, что пошло не так? |
То есть что я вчера спрашивал, работает как я сделал вначале? |
Какую проблему решает ваш PR:
Слегка изменил внутреннее устройство класса
Keyboard
. На мысль натолкнул issue #594. В принципе, в текущей версии VK API нет такой ситуации, когда пользователь хотел бы передать пустую строку в клавиатуру, но текущая имплементация все равно позволяет это делать.Данное изменение добавляет возможность вызывать
keyboard.row()
сколько угодно раз подряд, а так же в самом конце, безвреда для здоровьясоздания лишних пустых строк.Связанные issue: #594