Skip to content

Improved codegen schema, deleted several DTO structures #9

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

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

Liiizak
Copy link
Contributor

@Liiizak Liiizak commented Jan 29, 2025


Note: by creating a PR or an issue you automatically agree to the CLA. See CONTRIBUTING.md. Feel free to remove this note, the agreement holds.

@Liiizak
Copy link
Contributor Author

Liiizak commented Mar 9, 2025

Была дописана схема и убраны некоторые структуры, замененные на кодогенерированные, но не все. Структуры Article и Comment в src/dto имеют поле типа userver::storages::postgres::TimePointTz. Можно заменить структуры, если кодогенерация позволяет делать такие поля, но я нашла только utils::datetime::TimePointTz. Profile используется в Article. Структура Filter имеет нестандартный парсер, поэтому ее тоже не получилось убрать
Пришлось изменить логику схемы: теперь при регистрации нужно задавать поля image и bio, так как создается объект пользователя, у которого эти поля являются обязательными (исходя из схемы https://github.com/gothinkster/realworld/blob/6dc657a136a00b4f6441a5b4506b7e5712c79ebd/api/openapi.yml#L451), и непонятно как создавать нового User'a, при этом с отсутствующими обязательными параметрами
Касательно упавшего докера: жду, пока починят здесь https://github.com/userver-framework/uservice-dynconf/actions/runs/12982716994/job/36202908876

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant