Update money-bot (money_bot)

radio-t / rt-bot

Мета-репо для движения "создай своего бота"

129 stars 46 forks source link

Update money-bot (money_bot) #184

Closed exelban closed 7 years ago

exelban commented 7 years ago

Изменение:

изменил название на money-bot;
теперь бот пишет от кого он конвертировал валюту;
добавил белорусский рубль;
изменил выдачу результата на таблицу.

exelban commented 7 years ago

Да все время не понимал почему падают тесты( Извиняюсь.

exelban commented 7 years ago

Зачем вообще нужны эти тесты? Если и так много ботов прописывают какой то regEx что бы прошло на 100%, типа .+ или [a-zA-Z0-9._-]+.

Только всегда проблемы от этих тестов.

umputun commented 7 years ago

тесты нужны для того, чтоб мне руками не проверять поднимется все после очередного 14 коммита или нет. Прописано ли все в nginx и в композе. Делает ли бот минимум из того, что он должен делать.

umputun commented 7 years ago

ты проигнорировал часть про \\n (два слэша) или сам js это делает?

umputun commented 7 years ago

ох, все, не только первый.

exelban commented 7 years ago

Может тупой вопрос, но какая разница между \\n и \n?

exelban commented 7 years ago

Понял, первый слэш это экранирование \n.