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 support for restricted privacy user forwards #7

Merged
merged 1 commit into from
Aug 31, 2021
Merged

Conversation

rus-ai
Copy link
Contributor

@rus-ai rus-ai commented Jul 15, 2021

Добавил способ сохранить реальный ID юзера, в случае если он запретил оставлять ссылку на его профиль при форвардинге.

Идеально было бы сделать базу данных, но мне хотелось без создания дополнительных сущностей.

Алгоритм работы:

  1. При форвардинге сообщения в чат поддержки бот проверяет запретил ли юзер делать на него ссылку, и если да то следом в чате поддержки сам отвечает на пересланное сообщение оставляя ID юзера и комментарий для команды поддержки (задается в .env).
  2. При ответе на форварднутое сообщение командой поддержки - проверяется скрыта ли ссылка на юзера и если да выдается заданное в .env сообщение об ошибке.
  3. При ответе командой поддержки на сообщение бота - из него извлекается ID пользователя и ответ уходит ему.

@rus-ai
Copy link
Contributor Author

rus-ai commented Jul 15, 2021

Закроет эту issue
#1

@Nuke142
Copy link

Nuke142 commented Jul 17, 2021

Ничего не изменилось. Перезавёл приложение в Heroku, удалив старое, но как все было, так и осталось - при запрете пересылки сообщений мне не приходят сообщения на мой другой аккаунт. Как только разрешаешь пересылку, то сообщения приходят.

@rus-ai
Copy link
Contributor Author

rus-ai commented Jul 30, 2021

@Nuke142 я не правил app.json и поэтому даже если нажимать кнопку Deploy в моей репозитории то ставится с исходников в этом основном репозитории. Сделано для того чтобы pull request в основную ветку сделать.

Я сделал сейчас отдельную ветку с исправленным деплоем. Вот отсюда должно нормально деплоится, у меня получилось. Если отключить форвардинг - то пересылается сообщение в чат поддержки и следом за ним сообщение бота с ИД пользователя и призывом ответить на это второе сообщение. Если все равно ответить на первое сообщение выдаст сообщение об ошибке.

Параметры env для кастомизаций сделал необязательными REPLY_TO_THIS_MESSAGE, WRONG_REPLY
У меня значения такие "Пользователь выше запретил пересылать его сообщения, чтобы ответить ему ответьте на это сообщение а не на его сообщение." и "Ответьте не на это сообщение, а на следующее" но можно вообще не задавать.

@Nuke142
Copy link

Nuke142 commented Aug 2, 2021

@rus-ai
по нажатию на кнопку из ветки deploy по ссылке сейчас ругается на app.json
4444

@rus-ai
Copy link
Contributor Author

rus-ai commented Aug 2, 2021

@Nuke142 да спасибо, делал параметры не обязательными и запятую не убрал. Сейчас должно работать.

@Nuke142
Copy link

Nuke142 commented Aug 2, 2021

Все работает теперь как надо! Проверил по-всякому.
Спасибо!

@Akiyamka
Copy link

А чего не мержите?

@ohld ohld merged commit 93782bf into ohld:main Aug 31, 2021
@ohld
Copy link
Owner

ohld commented Aug 31, 2021

Спасибо! Ищью закрывает

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

Successfully merging this pull request may close these issues.

4 participants