search

hse-wasd-team / werec

3 stars 0 forks source link

Add specificaion 💩 #31

Closed ramilyayus closed 2 years ago

ramilyayus commented 2 years ago

18

zhurbey commented 2 years ago

может добавим префикс к ручкам, что-то вроде /api/v1/<>

zhurbey commented 2 years ago

пара коротких замечаний в воздух:

  1. определенно возвращать комменты в ручке get /feed идея довольно странная, так точно лучше не делать, а выделить этот кусок в отдельную ручку, в ней прикрутить логику пагинации. Конфиги наверное стоит все-таки подтягивать, так как чаще всего могут требоваться вместе. Вообще это касается всех мест где возвращаются фиды
  2. оборачивать данные в ответах стоит в {“data”: {и тут ответ}}. Так стоит делать во всех ручках, в том числе в запросах на сервак
  3. rating -> review
  4. думаю что в put запросах на фиды не стоит прикладывать конфигурации, в них редачить только метаинфу самих фидов, на конфиги сделать отдельную ручку. Хотя честно не уверен, возможно в каких-то сценариях так будет и правда проще, но такая логика bulk update’а все равно скорее всего должна быть в другом месте

смотрел не сильно детально и думаю что когда будем писать код что-то все равно будет сильно меняться, по этим коментам тоже ничего править не нужно. Го вливать)

@ramilyayus