search

strelka-skaut / api-spec

0 stars 0 forks source link

proto pages #1

Open martin-heralecky opened 2 years ago

martin-heralecky commented 2 years ago

https://github.com/strelka-skaut/api-spec/blob/46deb9f50ee8f5453cdcf559b76cf8bc2a668494/protobuf/pages.proto#L82 proc ne jenom id? jestli page_id, tak by melo byt i page_name, page_slug apod. vsude v PageService bych nahradil page_id za id.

https://github.com/strelka-skaut/api-spec/blob/46deb9f50ee8f5453cdcf559b76cf8bc2a668494/protobuf/pages.proto#L86 to je k cemu?

MrVoltz commented 2 years ago

O 1. jsme se už bavili, nevím proč jsi to tak začal dělat, ale moje zdůvodnění je, že kdekoliv se filtruje, tak píšu page_id. U 2. musíme umět rozlišit, kdy chceš updatovat site_id na null a kdy chceš nechat původní hodnotu.