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

522 routes modification #580

Open
wants to merge 27 commits into
base: master
Choose a base branch
from
Open

522 routes modification #580

wants to merge 27 commits into from

Conversation

necitboss
Copy link

Задача #522

  1. Blueprint доставил немало проблем и требует изменения других файлов. В основном, надо во всех файлах .py и .html искать "url_for(<ссылка>)" и менять ссылку в соответствии со следующим правилом: .<метод>, где на месте blueprint должен быть название соответствующего blueprint, а метод - тот метод, который используется для обработки запроса.
  2. Также методы тоже пришлось переименовать, т.к. при совпадении метода с названием blueprint возникает ошибка, поэтому в таких случаях в названия функций я добавил "_main".
  3. Ну и по мелочи: вынес константы из server.py в server_consts.py, изменил fetch в check_list.js с "get_csv" + "?" + params на "../get_csv" + "?" + params, т.к. с blueprint необходимо выходить из папки, т.к. запрос считается не от корня проекта, а от текущего "каталога".

@necitboss necitboss reopened this Jul 3, 2024
@necitboss necitboss closed this Jul 3, 2024
@necitboss necitboss reopened this Jul 3, 2024
@necitboss necitboss closed this Jul 3, 2024
@necitboss necitboss reopened this Jul 3, 2024
@necitboss necitboss closed this Jul 3, 2024
@necitboss necitboss reopened this Jul 3, 2024
@necitboss necitboss closed this Jul 3, 2024
@necitboss necitboss reopened this Jul 3, 2024
@HadronCollider HadronCollider added the future_integration Интеграция/мерж в будущих релизах label Jul 22, 2024
@github-actions github-actions bot added the has conflicts if new merge has conflicts label Aug 26, 2024
@github-actions github-actions bot removed the has conflicts if new merge has conflicts label Aug 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
future_integration Интеграция/мерж в будущих релизах
Projects
None yet
2 participants