add: addChat/configRaffle; fix: winnerMessage/raffleMessage #34

2022-01-15T14:42:23Z

sodapng commented

2022-01-15 14:42:23 +00:00

(Migrated from github.com)

Настроены команды addChat и configRaffle
Обработка исключения бота из чата или канала.
Исправлены команды winnerMessage и raffleMessage для каналов

- Настроены команды addChat и configRaffle - Обработка исключения бота из чата или канала. - Исправлены команды winnerMessage и raffleMessage для каналов

backmeupplz (Migrated from github.com) requested changes

2022-01-15 15:43:59 +00:00

locales/en.yaml

					
				@ -41,7 +41,25 @@ custom_winner_message: Reply to this message to setup a custom winner message. M

				no_winner_message: Randy will use standard winner message now.

Great! Now you're configuring this chat.

Great! Now you're configuring this chat: ${chat}.

locales/ru.yaml

					
				@ -41,7 +41,25 @@ custom_winner_message: Ответьте на это сообщение, чтоб

				no_winner_message: Теперь Ренди будет использовать стандартное сообщение о победе.

Отличается от en

src/app.ts

Сверху — миддлвари для всех последующих запросов. Хочешь сделать фильтр — делай его отдельно

Вместо этого, стоит просто запрашивать и проверять getChatMember, чтобы лишний раз не запоминать админов. Мы не так часто будем запрашивать статус мембера — поэтому это ок.

Вместо этого, стоит просто запрашивать и проверять `getChatMember`, чтобы лишний раз не запоминать админов. Мы не так часто будем запрашивать статус мембера — поэтому это ок.

removed

`removed`

// Inline menu callbacks

`// Inline menu callbacks`

src/filters/gotKickedMe.ts

onlyKickedMe

`onlyKickedMe`

Какие еще статусы есть? Мб kicked или restricted?

Какие еще статусы есть? Мб `kicked` или `restricted`?

src/handlers/configRaffle.ts

					
				@ -1,98 +1,26 @@

				// import { ContextMessageUpdate, Telegraf } from 'telegraf'

				// import { findChat } from '../models/Chat'

				// import { loc } from '../helpers/locale'

				import { InlineKeyboard } from 'grammy'

А у нас есть middleware типа onlyPrivateChats?

А у нас есть middleware типа `onlyPrivateChats`?

Если нет, давай сделаем

Это же chat, а не me. me — это сам бот

Это же `chat`, а не `me`. `me` — это сам бот

Лучше ответь ошибкой

src/helpers/deregisterAdminChatIds.ts

Не пушь лишние console.log в PR

Не пушь лишние `console.log` в PR

removeChatFromConfigurableChats

`removeChatFromConfigurableChats`

Используй mongo query с $pull, чтобы убрать ctx.chat.id из adminChatIds всех пользователей, у которых он есть.

Используй mongo query с `$pull`, чтобы убрать `ctx.chat.id` из `adminChatIds` всех пользователей, у которых он есть.

src/helpers/setEditedChat.ts

					
				@ -0,0 +1,43 @@

				import {

Стоит ответить ошибкой

src/middlewares/checkEditedChatId.ts

onlyWithEditedChatId

`onlyWithEditedChatId`

src/models/Chat.ts

Скорее всего, это не нужно

sodapng (Migrated from github.com) reviewed

2022-01-15 17:32:14 +00:00

src/filters/gotKickedMe.ts

https://core.telegram.org/bots/api#chatmemberbanned

kicked присутствует только в old_chat_member.status - старое состояние, необходимое для определения того, что member был когда-то в чате/канале.

https://core.telegram.org/bots/api#chatmemberbanned `kicked` присутствует только в `old_chat_member.status` - старое состояние, необходимое для определения того, что `member` был когда-то в чате/канале.

sodapng (Migrated from github.com) reviewed

2022-01-15 18:27:43 +00:00

src/handlers/configRaffle.ts

					
				@ -1,98 +1,26 @@

				// import { ContextMessageUpdate, Telegraf } from 'telegraf'

				// import { findChat } from '../models/Chat'

				// import { loc } from '../helpers/locale'

				import { InlineKeyboard } from 'grammy'

У private chat отсутствует свойство title
Если не проверять тип, то будет выдавать ошибку:
Свойство "title" не существует в типе "PrivateGetChat".

У `private chat` отсутствует свойство `title` Если не проверять тип, то будет выдавать ошибку: `Свойство "title" не существует в типе "PrivateGetChat".`

sodapng (Migrated from github.com) reviewed

2022-01-15 20:14:53 +00:00

src/filters/gotKickedMe.ts

Оказывается для type channel status === left, для type group/supergroup status === kicked. Хотя условие ctx.myChatMember?.new_chat_member.status === 'left' работало и для группы.

Оказывается для `type channel status === left, для type group/supergroup status === kicked`. Хотя условие `ctx.myChatMember?.new_chat_member.status === 'left'` работало и для группы.

sodapng (Migrated from github.com) reviewed

2022-01-15 20:22:25 +00:00

src/app.ts

Здесь я пытался добавлять ctx.chat.id в adminChatIds.
Уже исправил на:

import { findOrCreateChat } from '@/models/Chat'
import Context from '@/models/Context'

export default async function addAdminChatIds(ctx: Context) {
  if (!ctx.chat || !ctx.from) return

  const { doc: chat } = await findOrCreateChat(ctx.from.id)

  chat.adminChatIds.push(ctx.chat.id)
  return chat.save()
}

Может быть реализовать это внутри middleware attachChat?

Здесь я пытался добавлять `ctx.chat.id` в `adminChatIds`. Уже исправил на: ``` import { findOrCreateChat } from '@/models/Chat' import Context from '@/models/Context' export default async function addAdminChatIds(ctx: Context) { if (!ctx.chat || !ctx.from) return const { doc: chat } = await findOrCreateChat(ctx.from.id) chat.adminChatIds.push(ctx.chat.id) return chat.save() } ``` Может быть реализовать это внутри `middleware attachChat`?

sodapng (Migrated from github.com) reviewed

2022-01-16 12:41:37 +00:00

src/filters/onlyKickedMe.ts

Исправлю в следующем PR.

sodapng (Migrated from github.com) reviewed

2022-01-16 12:43:43 +00:00

src/middlewares/botRemovedFromAdminChatIds.ts

Это чтобы не дублировать код в src/helpers/setEditedChat.ts

Это чтобы не дублировать код в `src/helpers/setEditedChat.ts`

backmeupplz (Migrated from github.com) requested changes

2022-01-16 16:03:28 +00:00

src/app.ts

В этом файле должно быть минимум изменений. Убери все изменения, которые не необходимы

src/filters/onlyKickedMe.ts

Исправь в этом

src/middlewares/attachChat.ts

Не пушь в PR закомментированный код

src/middlewares/botRemovedFromAdminChatIds.ts

Не дублируй, но сделай функцию типа getChatIdFromCallbackQuery

Не дублируй, но сделай функцию типа `getChatIdFromCallbackQuery`

Явно не нужно глобальный контекст закидывать лишним :)

No chat id in callback query

`No chat id in callback query`

src/middlewares/onlyWithEditedChatId.ts

Думаю, можно сделать !ctx.dbchat.editedChatId

Думаю, можно сделать `!ctx.dbchat.editedChatId`

src/models/Chat.ts

					
				@ -52,3 +52,73 @@ const ChatModel = getModelForClass(Chat)

				export function findOrCreateChat(chatId: number) {

Ты делаешь delete, а не update

Ты делаешь `delete`, а не `update`

Найди чаты, где { adminChatIds: chatId }, а не делай апдейт всем чатам

Найди чаты, где `{ adminChatIds: chatId }`, а не делай апдейт всем чатам

$eq можно опустить, он подразумевается же

`$eq` можно опустить, он подразумевается же

src/models/Context.ts

Не думаю, что нам нужен тут chatId. Мы его можем получить из dbchat

Не думаю, что нам нужен тут `chatId`. Мы его можем получить из `dbchat`

backmeupplz (Migrated from github.com) requested changes

2022-01-16 20:33:25 +00:00

src/app.ts

Лишнее изменение

// Added bot as admin

`// Added bot as admin`

// Kicked or removed bot

`// Kicked or removed bot`

src/app.ts

					
				@ -76,0 +93,4 @@

				  bot.command(

				    'noCustomWinnerMessage',

				    onlyWithEditedChatIdOrPublicChat,

				    handleNoCustomWinnerMessage

Лишнее изменение

src/filters/onlyIfBotDesignatedAdmin.ts

Только если бот был админом — и его снова сделали админом?

src/filters/onlyKickedMe.ts

onlyKickedOrRemovedAdminMe

`onlyKickedOrRemovedAdminMe`

src/filters/onlyRepliesToBots.ts

onlyRepliesToBotsOrChannelPosts

`onlyRepliesToBotsOrChannelPosts`

src/handlers/addChat.ts

					
				@ -1,45 +1,66 @@

				// import { ContextMessageUpdate, Telegraf } from 'telegraf'

				// import { findChat } from '../models/Chat'

				// import { loc } from '../helpers/locale'

				import { GrammyError } from 'grammy'

Верни ошибку, если нет ctx.match

Верни ошибку, если нет `ctx.match`

А если закинут уже /addChat -123456?

А если закинут уже `/addChat -123456`?

Лучше сделать через ?, чтобы тип не нужно было указывать

Лучше сделать через `?`, чтобы тип не нужно было указывать

Можно вернуть 21 строку сразу

Это ненужная строка

src/handlers/configRaffle.ts

					
				@ -1,98 +1,26 @@

				// import { ContextMessageUpdate, Telegraf } from 'telegraf'

				// import { findChat } from '../models/Chat'

				// import { loc } from '../helpers/locale'

				import { InlineKeyboard } from 'grammy'

Используй тут for of (for (const chatId of ctx.dbchat.adminChatIds) {})

Используй тут `for of` (`for (const chatId of ctx.dbchat.adminChatIds) {}`)

Здесь нужно показать пользователю ошибку, а не просто молча заканчивать выполнение функции

src/helpers/addChatFromConfigurableChats.ts

AddChatToAdminChatIds

`AddChatToAdminChatIds`

src/middlewares/botRemovedFromAdminChatIds.ts

Это надо проверить внутри хендлера коллбека, не нужно это выносить в отдельную мидлварю

src/middlewares/onlyWithEditedChatId.ts

onlyWithEditedChatIdOrPublicChat

`onlyWithEditedChatIdOrPublicChat`

src/models/Chat.ts

					
				@ -52,3 +52,73 @@ const ChatModel = getModelForClass(Chat)

				export function findOrCreateChat(chatId: number) {

Тут должен быть 1 или хотя бы true

Тут должен быть `1` или хотя бы `true`

deleteFromAdminChatIds

`deleteFromAdminChatIds`

deleteFromEditedChatId

`deleteFromEditedChatId`

sodapng (Migrated from github.com) reviewed

2022-01-17 12:00:49 +00:00

src/filters/onlyIfBotDesignatedAdmin.ts

Ограничили в правах. Можно же изменить права (изменение профиля группы, удаление сообщений и т.д.) админов. Если оставить только ctx.myChatMember?.new_chat_member.status === 'administrator', то chat.id будет дублироваться в adminChatIds.

Ограничили в правах. Можно же изменить права (изменение профиля группы, удаление сообщений и т.д.) админов. Если оставить только `ctx.myChatMember?.new_chat_member.status === 'administrator'`, то `chat.id` будет дублироваться в `adminChatIds`.

backmeupplz (Migrated from github.com) requested changes

2022-01-18 17:15:54 +00:00

src/app.ts

					
				@ -57,26 +66,56 @@ async function runApp() {

				    .use(numberOfWinnersMenu)

Наверное, это должно быть chooseChatToCongigure? 🤔 chooseChatOrChannelToConfigure?

Наверное, это должно быть `chooseChatToCongigure`? 🤔 `chooseChatOrChannelToConfigure`?

не стоит функции с большой буквы называть

Нужно проверить случай, когда добавили админом с достаточными правами или когда выдали достаточно прав

src/app.ts

					
				@ -76,0 +108,4 @@

				  bot.command(

				    'checkSubscription',

				    onlyWithEditedChatIdOrPublicChat,

				    handleCheckSubscription

Нужно проверить случай, когда кикнули или лишили необходимых прав

src/filters/onlyIfBotDesignatedAdmin.ts

adminChatIds никогда не должен дублироваться, добавление\удаление должно быть через mongo query, которое считает adminChatIds сетом

`adminChatIds` никогда не должен дублироваться, добавление\удаление должно быть через mongo query, которое считает `adminChatIds` сетом

Проверь тут сразу и права бота, стоит это сделать

src/handlers/addChat.ts

					
				@ -1,45 +1,66 @@

				// import { ContextMessageUpdate, Telegraf } from 'telegraf'

				// import { findChat } from '../models/Chat'

				// import { loc } from '../helpers/locale'

				import { GrammyError } from 'grammy'

Нужно юзеру показать ошибку, не надо молчать

Нужно проверить, достаточно ли прав

src/helpers/AddChatToAdminChatIds.ts

Я так понял, ты добавляешь бота в конфигурируемые чаты пользователю, когда пользователь дает админку боту? Хороший ход

src/helpers/removeChatFromConfigurableChats.ts

					
				@ -0,0 +1,9 @@

				import { deleteFromAdminChatIds, deleteFromEditedChatId } from '@/models/Chat'

Давай мб экспепшн кинем?

sodapng (Migrated from github.com) reviewed

2022-01-19 19:38:50 +00:00

src/helpers/removeChatFromConfigurableChats.ts

					
				@ -0,0 +1,9 @@

				import { deleteFromAdminChatIds, deleteFromEditedChatId } from '@/models/Chat'

Добавил! Можно узнать, зачем мы это делаем? Мы заранее подстраховываемся, если что-то пойдет не так?

backmeupplz (Migrated from github.com) requested changes

2022-01-20 03:18:38 +00:00

src/filters/onlyIfBotDesignatedAdmin.ts

onlyIfGotEnoughPermissions

`onlyIfGotEnoughPermissions`

src/handlers/addChat.ts

					
				@ -46,0 +58,4 @@

				      await addChatIdFromAdminChatIds(ctx.from.id, chat.id)

				      return ctx.replyWithLocalization('config_raffle_instructions')

				    }

Этот хак с превращением ctx.match обязателен? Это же и так должна быть строка?

Этот хак с превращением `ctx.match` обязателен? Это же и так должна быть строка?

src/handlers/addChat.ts

Давай еще проверим права человека, который вызывает эту команду

src/helpers/AddChatToAdminChatIds.ts

Имя файла тоже нужно написать с маленькой буквы. А потом git rm -r --cached . && git add .

Имя файла тоже нужно написать с маленькой буквы. А потом `git rm -r --cached . && git add .`

src/helpers/getChatIdFromCallbackQuery.ts

					
				@ -0,0 +1,11 @@

				import Context from '@/models/Context'

Давай вот тут сразу конвертнем id: ... = +ctx.callbackQuery?.data?.replace('chat:', '')

Давай вот тут сразу конвертнем id: `... = +ctx.callbackQuery?.data?.replace('chat:', '')`

А вот тут давай проверим, не NaN ли chatId, если NaN, то кинем эксепшен

А вот тут давай проверим, не `NaN` ли `chatId`, если `NaN`, то кинем эксепшен

src/helpers/setEditedChat.ts

					
				@ -0,0 +1,43 @@

				import {

adminChatIds — это список айди чатов, которые человек (в приватном чате) может конфигурировать. Проверять тут нужно через getChatMember бота, является ли он все еще админом. Если нет — кинуть ошибку и убрать чат из adminChatIds

`adminChatIds` — это список айди чатов, которые человек (в приватном чате) может конфигурировать. Проверять тут нужно через `getChatMember` бота, является ли он все еще админом. Если нет — кинуть ошибку и убрать чат из `adminChatIds`

sodapng (Migrated from github.com) reviewed

2022-01-20 10:54:10 +00:00

src/handlers/addChat.ts

					
				@ -46,0 +58,4 @@

				      await addChatIdFromAdminChatIds(ctx.from.id, chat.id)

				      return ctx.replyWithLocalization('config_raffle_instructions')

				    }

(property) Context.match?: string | RegExpMatchArray

`(property) Context.match?: string | RegExpMatchArray`

backmeupplz (Migrated from github.com) requested changes

2022-01-21 17:01:15 +00:00

src/handlers/addChat.ts

					
				@ -1,45 +1,66 @@

				// import { ContextMessageUpdate, Telegraf } from 'telegraf'

				// import { findChat } from '../models/Chat'

				// import { loc } from '../helpers/locale'

				import { GrammyError } from 'grammy'

Не isChatAdmin, а userChatMemberStatus

Не `isChatAdmin`, а `userChatMemberStatus`

src/helpers/setEditedChat.ts

					
				@ -0,0 +1,43 @@

				import {

Это userChatMemberStatus

Это `userChatMemberStatus`

Sign in to join this conversation.

No reviewers

backmeupplz

No labels

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

backmeupplz/randymbot!34

				`@ -41,7 +41,25 @@ custom_winner_message: Reply to this message to setup a custom winner message. M`
				`no_winner_message: Randy will use standard winner message now.`

				`@ -41,7 +41,25 @@ custom_winner_message: Ответьте на это сообщение, чтоб`
				`no_winner_message: Теперь Ренди будет использовать стандартное сообщение о победе.`

				`@ -52,3 +52,73 @@ const ChatModel = getModelForClass(Chat)`
				`export function findOrCreateChat(chatId: number) {`

				`@ -57,26 +66,56 @@ async function runApp() {`
				`.use(numberOfWinnersMenu)`

				`@ -0,0 +1,9 @@`
				`import { deleteFromAdminChatIds, deleteFromEditedChatId } from '@/models/Chat'`

Rows
Columns