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

Реакция бота на нового юзера #7

Closed
sergeysova opened this issue Nov 2, 2017 · 4 comments
Closed

Реакция бота на нового юзера #7

sergeysova opened this issue Nov 2, 2017 · 4 comments

Comments

@sergeysova
Copy link
Member

При входе юзера в чат проверять его наличие в черном списке.

Если он там есть, сразу же банить и удалять его сообщения, если есть

@sergeysova sergeysova added this to the v0.5 milestone Nov 4, 2017
@sergeysova sergeysova changed the title Реакции бота Реакция бота на нового юзера Nov 4, 2017
@dmitryshelomanov
Copy link
Member

Вроде сделал. Залью позже. хз как. Не просто же комитить. cz нету ? А и я не уверен работает ли удаление месаг.

@sergeysova
Copy link
Member Author

Надо протестить эту фичу и #3

@dmitryshelomanov
Copy link
Member

dmitryshelomanov commented Nov 30, 2017

@sergeysova затестил. И тут напишу D

  • Не смертельная трабла

удаляя сообщения в телеге у нас в бд оно остается и в следующий раз телега вернет 404 и сколько лишних запросов будет (мб удалять и в бд) ?

  • Смертельная

Тут featureBotParticipation нету notifySpammerAutoban ибо откуда ему там быть ? Ты планировал футурам биндить контекст ? А так пока его там нету D
А в общем все работает. Я затестил ручками D. Хз как там будет с твоими тестами. Но я их жаждю. Ибо жесть. Хотя так оно позволит тестить ?

@dmitryshelomanov
Copy link
Member

upd
ctx.privateChannel.notifySpammerAutoban смертельная трабла снята
просто видимо ты ночью писал D
нужно было из контекста брать

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

No branches or pull requests

2 participants