Skip to content
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

add README on English #2

Closed
lgg opened this issue Dec 30, 2021 · 8 comments
Closed

add README on English #2

lgg opened this issue Dec 30, 2021 · 8 comments

Comments

@lgg
Copy link

lgg commented Dec 30, 2021

I can create PR, will u accept it?

@lgg
Copy link
Author

lgg commented Dec 30, 2021

я бы ещё добавил информацию:

  • где брать chat id и что для групп его важно указывать с -
  • docker-compose сразу с environment прописанными

ну и следом PR на i18n надо бы сделать)

@MasterGroosha
Copy link
Owner

docker-compose сразу с environment прописанными

Не самая лучшая идея, на мой взгляд. То, как сделано сейчас, вполне нормально.

где брать chat id и что для групп его важно указывать с -

Добавлю в ридми, если этого там ещё нет

@lgg
Copy link
Author

lgg commented Dec 30, 2021

@MasterGroosha
Во-первых, спасибо огромное за бота! Если бы не он перед новогодними - совсем с ума сошёл бы. Автор - крепкого тебе здоровья и долгих лет жизни!

Не самая лучшая идея, на мой взгляд. То, как сделано сейчас, вполне нормально.

не возражаю, оке

Добавлю в ридми, если этого там ещё нет
Я делал по: https://stackoverflow.com/questions/32423837/telegram-bot-how-to-get-a-group-chat-id

До этого супер плотно не работал и не сразу понял, что для группы id начинается с -, потом вспомнил как было в ВК и стало очевидно, но если не против - то лучше такое в README добавить:

Переменная ADMIN_CHAT_ID должна быть числом, равным идентификатору либо лички с админом, либо группы (желательно, супергруппы), все участники которой будут считаться равноправными администраторами бота. ID групп в телеграм начинаются с `-`

И на счёт англ ридми - ок, если его добавлю, примете Pull Request?

@MasterGroosha
Copy link
Owner

то лучше такое в README добавить

Справедливо. Только, наверное, не в README, а в env_dist, как раз там, где указывается переменная ADMIN_CHAT_ID.

И на счёт англ ридми - ок, если его добавлю, примете Pull Request?

Так вышло, что основная разработка у меня ведётся на личном GitLab, а сюда просто зеркалируется код. Можешь создать Pull Request, но изменения оттуда я буду сливать вручную и пушить "от себя" (надо подумать, как сохранить список контрибуторов).

У меня при этом возникает другой вопрос, более глобальный: а зачем README на английском? Пардон за вопрос, но я реально хочу понять причину сего действия. Ибо если планируется показывать код иностранцам, то есть смысл мне полностью обновить комментарии в самом коде, переведя их на английский, как в других моих проектах.

@lgg
Copy link
Author

lgg commented Dec 30, 2021

Только, наверное, не в README, а в env_dist, как раз там, где указывается переменная ADMIN_CHAT_ID

+++

но изменения оттуда я буду сливать вручную и пушить "от себя" (надо подумать, как сохранить список контрибуторов).

Можно не сохранять

Ибо если планируется показывать код иностранцам, то есть смысл мне полностью обновить комментарии в самом коде, переведя их на английский, как в других моих проектах.

На самом деле мотивация - "душа болит". Я наткнулся на этот проект и он пушка, но тут описание только на русском, что сразу же значительно срезает потенциальную целевую аудиторию. У меня есть скромный неработающий бот почти 3х летней давности (https://github.com/lgg/vk-to-telegram-bot) - так мне до сих пор некоторые иностранцы даже в телеграм в личку стучатся и что-то спрашивают про него и в выдаче он где-то мелькает. То есть лично для меня - мотивацию можно сформулировать "сделать мир лучше" - я наткнулся на этот проект, сам им воспользовался и уверен, что в мире полно людей, кому он нужен и может быть полезным, но из-за языкового барьера многие даже банально не найдут этого бота или пройдут мимо.

Я уже после праздников закину перевод, а там посмотрим. В идеале, если вообще сможем/сможешь сделать обновление, где все текстовые переменные (ответы бота + вроде где-то ещё видео) вынесены в отдельный файл, чтобы можно было делать мультиязычность за счёт редактирования этого файла + в целом это сделает бот более универсальным, так как мне для использования как раз понадобилось только закомментить удаление ответа бота и поменять текста, больше ничего не трогал и всё запустилось "из коробки".

@lgg
Copy link
Author

lgg commented Dec 30, 2021

Ни в коем случае не гонюсь за звездочками, наоборот, для меня они показатель - что проект действительно кому-то полезен и нужен.

например, репка - узкоспециализированный сборник ссылок, но за счёт английского языка уже почти 150+ звездочек = 150+ человек, которые им воспользовались по назначению и кому это помогло/съэкономило время.

Ставлю, что если в данном боте добавить README на английском + причесать оформление репки на англ (описание и тд) - то народу сюда повалит уйма и через год будет уже под 1000 активных юзеров.

@MasterGroosha
Copy link
Owner

@lgg Спасибо за подробное описание!
Я думаю, на первой итерации можно сделать переводы в одном JSON-файле, как в другом моём боте.
Полноценный i18n на gettext делать не хочу принципиально, но думаю над тем, как бы прикрутить Fluent от Mozilla (есть демка, осталось в реальный проект перенести).

то народу сюда повалит уйма и через год будет уже под 1000 активных юзеров

Ха! Это заблуждение, проверено :)

@MasterGroosha
Copy link
Owner

Добавил предупреждение об отрицательных ID у групп и каналов (и о способе их получения) в f34fd17

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants