4
0
Fork 0
Wiki Issues 6 Projects Code Pull Requests Releases Activity
Labels Milestones issues_tree

adminpanel_backend #6

Closed
Fynjy wants to merge 5 commits from adminpanel_backend into master
Conversation 2 Commits 5 Files Changed 20
Fynjy commented 1 year ago
Collaborator
There is no content yet.
Fynjy added 4 commits 1 year ago
Bezborodov commented 1 year ago
Owner

Необходимо полностью переделать файл mod_table_operate.py

  1. Не нужно ничего нового импортировать типа import os import sqlite3
    ...
    Все запрос ык БД сделаны через self.m_Bot.SQLRequest
    Этот код ничего не должен занчть какая БД использвется.
  2. from bot_sys.config import g_telegram_bot_api_token
    не нужен. У self.m_Bot есть по моему Get_API_Token()
  3. Функции async def create_order(): и my_handler()
    Это никуда не годится!
    Этот код универсальный для всех модулей. Он совршенно не должен знать про какие-то там "заказы"
    Уверен что если мы будем использовать другие модули, то этот код сломается.

Насколько я понял ты просто пытаешься сохранить две картинки по заказу и по чеку. Ну так так бы и назвал функции! А то create_order и my_handler совсем непонятно к чему.

И ещё. Ты сохраняешь картинки в локальную папку. А это нужно? Может быть сразу показывать ссылку на картинку по адресу - https://api.telegram.org/bot{token}/getFile?file_id={cheque} ?
Пусть она каждый раз запрашивается при просмотре. Зачем её сохранять себе?

Ещё из мелочей
file_id = param[2]
Это неправильно! У меня это делается через ID - self.m_Table.GetFieldIDByDestiny

Полагаю что mod_table_operate.py не должен менять вообще для админки.

Необходимо полностью переделать файл mod_table_operate.py 1. Не нужно ничего нового импортировать типа import os import sqlite3 ... Все запрос ык БД сделаны через self.m_Bot.SQLRequest Этот код ничего не должен занчть какая БД использвется. 2. from bot_sys.config import g_telegram_bot_api_token не нужен. У self.m_Bot есть по моему Get_API_Token() 3. Функции async def create_order(): и my_handler() Это никуда не годится! Этот код универсальный для всех модулей. Он совршенно не должен знать про какие-то там "заказы" Уверен что если мы будем использовать другие модули, то этот код сломается. Насколько я понял ты просто пытаешься сохранить две картинки по заказу и по чеку. Ну так так бы и назвал функции! А то create_order и my_handler совсем непонятно к чему. И ещё. Ты сохраняешь картинки в локальную папку. А это нужно? Может быть сразу показывать ссылку на картинку по адресу - https://api.telegram.org/bot{token}/getFile?file_id={cheque} ? Пусть она каждый раз запрашивается при просмотре. Зачем её сохранять себе? Ещё из мелочей file_id = param[2] Это неправильно! У меня это делается через ID - self.m_Table.GetFieldIDByDestiny Полагаю что mod_table_operate.py не должен менять вообще для админки.
Bezborodov added 1 commit 1 year ago
Bezborodov commented 1 year ago
Owner

Закрываю этот, нужно создать новый

Закрываю этот, нужно создать новый
Bezborodov closed this pull request 1 year ago
This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
Request review
No reviewers
Labels
Apply labels
Clear labels
Внешняя разработка
Внешняя разработка Для добровольцев
Для добровольцев Задачи заказчиков
Задачи от заказчиков Техподдержка
Техподдержка
No Label Внешняя разработка Для добровольцев Задачи заказчиков Техподдержка
Milestone
Set milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Assign users
Clear assignees
No Assignees
2 Participants
Notifications
Due Date

No due date set.

issues.parent.title

issues.parent.pr_no_parents

Dependencies

This pull request currently doesn't have any dependencies.

Write Preview
Loading…
Cancel
Save
There is no content yet.
Delete Branch 'adminpanel_backend'

Deleting a branch is permanent. It CANNOT be undone. Continue?

No
Yes