fix kicked for admin; refactor; #35

2022-01-25T16:55:38Z

sodapng commented

2022-01-25 16:55:38 +00:00

(Migrated from github.com)

No description provided.

backmeupplz (Migrated from github.com) requested changes

2022-01-27 15:50:53 +00:00

locales/en.yaml

					
				@ -13,7 +13,7 @@ help: |

				  /noCustomRaffleMessage — remove the custom raffle message and use the default one

Давай поправим и само описание справа. Плюс, давай поправим русский, если нужно

src/app.ts

А что делает эта часть?

src/app.ts

Есть какие-то, какие нам не нужны? Давай сразу уберем

src/app.ts

Что за 500? Можно больше\меньше?

Что за `500`? Можно больше\меньше?

src/filters/onlyIfGotEnoughPermissions.ts

					
				@ -1,22 +1,26 @@

				import Context from '@/models/Context'

Не скоращай зазря. Твой код чаще читают, чем ты его пишешь. Пиши код понятно.

src/filters/onlyKickedAdmin.ts

Если есть логика, завязанная на том, бот это или нет — то нужно это учесть в названии функции. Представь, что кто-то потом эту функцию использует и начинает рвать на голове волосы, ведь не понимает, почем onlyKickedAdmin возвращает false, если кикают ботов.

Если есть логика, завязанная на том, бот это или нет — то нужно это учесть в названии функции. Представь, что кто-то потом эту функцию использует и начинает рвать на голове волосы, ведь не понимает, почем `onlyKickedAdmin` возвращает `false`, если кикают ботов.

src/handlers/addChat.ts

				`@ -50,17 +32,32 @@ export default async function handleAddChat(ctx: Context) {`

Либо ставь пробелы, либо не ставь их, но поддерживай consistency в стиле кода

sodapng (Migrated from github.com) reviewed

2022-01-27 16:52:32 +00:00

src/app.ts

@param concurrency Maximal number of updates to process concurrently По умолчанию 500.

run
Runs a grammY bot with long polling. Updates are processed concurrently with a default maximum concurrency of 500 updates. Calls to getUpdates will be slowed down and the limit parameter will be adjusted as soon as this load limit is reached.
You should use this method if your bot processes a lot of updates (several thousand per hour), or if your bot has long-running operations such as large file transfers.

`@param concurrency Maximal number of updates to process concurrently` По умолчанию 500. > `run` Runs a grammY bot with long polling. Updates are processed concurrently with a default maximum concurrency of 500 updates. Calls to getUpdates will be slowed down and the limit parameter will be adjusted as soon as this load limit is reached. You should use this method if your bot processes a lot of updates (several thousand per hour), or if your bot has long-running operations such as large file transfers.