abnormalize0 / DailyTeleFrog

0 stars 0 forks source link

#140 counter #143

Open skolbasin opened 2 months ago

skolbasin commented 2 months ago

Что сделал:

Дополнение:

  1. в документации есть расхождения с кодом в: 1.1. api_atricle_get. в параметрах 'container': 'header', а в коде 'container': ' body' 1.2. article_data_get. все параметры с 'is_required': False, а в коде 'is_required':True.
  2. не могу разобраться с запуском локально БД. так понимаю у нас пока mysql подключена. мне достаточно создать в корне проекта файл "db" и всё? или что-то еще требуется? и пчм мы явно укащваем порт 4321?
  3. не очень понял задание с добавлением ключей для открытия и просмотра статей
  4. не трогал пока задание с добавлением ключа уникального id пользователя при регистрации
  5. тесты и документацию пока не делал по твоего ревью по сделанной работе. Если смерджишь изменения, то создам ветку от orm-make-docker-venv и начну писать тесты и исправлю документацию. либо сделаю как скажешь
skolbasin commented 2 months ago

Да, у меня как раз view_article и open_article отвечают за это. Либо добавляют, либо выдадут None. Тоже задался вопросом их необходимости, но сделал по аналогии с лайками, т.к. данные функции уходят потом в backend.py в get_article Удалил is_open и is_views и удалил их инициализацию из get_article. Также в предпоследнем коммите внес в api.py в api_article_data_post вызов функций view_article и open_article