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.

Sign up for GitHub

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

Jump to bottom

Закастомлено редактирование сообщений бота в админке, удален файл с константами сообщений бота, константы заменены на вызов корутины получения сообщений бота из БД. #217

Merged
merged 9 commits into from
Dec 11, 2023
Merged

Закастомлено редактирование сообщений бота в админке, удален файл с константами сообщений бота, константы заменены на вызов корутины получения сообщений бота из БД. #217

merged 9 commits into from
Dec 11, 2023

Conversation

VladislavYar
Copy link
Contributor

@VladislavYar VladislavYar commented Dec 7, 2023 •
edited
Loading

Description

Закастомлено редактирование сообщений бота в админке, удален файл с константами сообщений бота, константы заменены на вызов корутины получения сообщений бота из БД.
Статику брал отсюда.

Type of change

Пожалуйста, удалите варианты, которые не относятся к ПР-у.

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Запустил бота, прошёлся по кнопкам.

Checklist:

  • Мой код соответствует code-style данного проекта
  • Я провел самоанализ собственного кода

@VladislavYar
Закастомлено редактирование сообщений бота в админке, удален файл с к…
e007aab 
…онстантами сообщений бота, константы заменены на вызов корутины получения сообщений бота из БД.
@VladislavYar
Починил тесты.
328191a
@VladislavYar VladislavYar linked an issue Dec 7, 2023 that may be closed by this pull request
Добавить визавиг markdown редактор #169
Closed
KonstantinRaikhert
KonstantinRaikhert reviewed Dec 8, 2023
View reviewed changes
Copy link
Member

@KonstantinRaikhert KonstantinRaikhert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

В итоге нам нужно статику добавлять?
И еще, по моему где-то в боте нужно было указать, чтобы апишка телеграма воспринимала markdown.
Пробовал менять через админку сообщение?

src/core/config/settings_base.py Outdated
@@ -30,6 +30,7 @@
"django.contrib.sessions",
"django.contrib.messages",
"django.contrib.staticfiles",
"ckeditor",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

давай его и остальные сразу перекинем в external_apps. Прорефакторим сразу

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Перенёс

@VladislavYar
Copy link
Contributor Author

VladislavYar commented Dec 9, 2023 •
edited
Loading

@KonstantinRaikhert да, действительно, надо было указать, не правильно протестил, смайлы изначально рабочие в этом редакторе, что логично, а форматирование текста нет.

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

Так же если пользователь, допустим, в никнейме добавит тег HTML, то это вызовет исключение.

KonstantinRaikhert
KonstantinRaikhert reviewed Dec 9, 2023
View reviewed changes
Copy link
Member

@KonstantinRaikhert KonstantinRaikhert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Так разве нам не надо добавлять статику в репозиторий?))

src/bot/handlers/conversation_handlers.py
@@ -333,6 +348,7 @@ async def send_both_users_message(
itspecialist.telegram_username,
guide_url,
),
parse_mode=ParseMode.HTML,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Я почему то думал что у него markdown)) Но это без разницы)

@KonstantinRaikhert
Copy link
Member

@KonstantinRaikhert да, действительно, надо было указать, не правильно протестил, смайлы изначально рабочие в этом редакторе, что логично, а форматирование текста нет.

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

Так же если пользователь, допустим, в никнейме добавит тег HTML, то это вызовет исключение.

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

А вот с пользователем штука интересная. То есть, если в username содержится HTML тег, то он не сможет воспользоваться ботом?

@VladislavYar
Copy link
Contributor Author

@KonstantinRaikhert да, действительно, надо было указать, не правильно протестил, смайлы изначально рабочие в этом редакторе, что логично, а форматирование текста нет.
На счёт статики. Перерыл инет, прямого упоминания возможности автоматической загрузки с какого-нибудь хранилища на сервер не нашёл, так же и с CDN.
Так же если пользователь, допустим, в никнейме добавит тег HTML, то это вызовет исключение.

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

А вот с пользователем штука интересная. То есть, если в username содержится HTML тег, то он не сможет воспользоваться ботом?

Ну я тогда в .gitignore напишу исключение.

Да, и если сам введёт тег, то не сможет.

@KonstantinRaikhert
Copy link
Member

Ну так наоборот статику нужно в репозиторий добавить. Или мы о разном?

Насчёт юзернейма волноваться не стоит
photo_2023-12-09_22-04-57

@VladislavYar
Copy link
Contributor Author

Ну так наоборот статику нужно в репозиторий добавить. Или мы о разном?

Насчёт юзернейма волноваться не стоит photo_2023-12-09_22-04-57

Ну я про сделать исключение для исключения, вхвхвх.

Про никнейм. Ну я же как-то ломал бота и сломал его минут 20 назад опять, когда проверял не с девелопа ли это идёт.
Я как понял, всё идёт от "parse_mode", т.е. у нас в боте вместо "{}" вставляет никнейм и если этот никнейм имеет не разрешенные(?) теги, то ломается отправка сообщения. Так же если админ решит вставить какой-нибудь тег в сообщение бота, случится тоже самое.

@VladislavYar
Copy link
Contributor Author

Ну так наоборот статику нужно в репозиторий добавить. Или мы о разном?

Насчёт юзернейма волноваться не стоит photo_2023-12-09_22-04-57

А, всё, понял. У нас немного путаница пошла. Я никнейм имею ввиду где имя и фамилия, а не где имя пользователя)

@KonstantinRaikhert
Copy link
Member

@VladislavYar В общем это конечно нюанс. а возможно будет проблема.. Может какой-то обработчик для имени пользователя и пароля придумать?.. Экранирование какое нибудь. В общем кину это в будущие таски. А эту таску давай закроем!)

@KonstantinRaikhert KonstantinRaikhert merged commit 96e67ce into develop Dec 11, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@KonstantinRaikhert KonstantinRaikhert KonstantinRaikhert approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Добавить визавиг markdown редактор
2 participants
@VladislavYar @KonstantinRaikhert