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.

Sign up for GitHub

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

Jump to bottom

Add associations #293

Draft
wants to merge 37 commits into
base: main
Choose a base branch
from
Draft

Add associations #293

wants to merge 37 commits into from

Conversation

HAEKADI
Copy link
Contributor

@HAEKADI HAEKADI commented Aug 31, 2023 •
edited
Loading

HAEKADI added 22 commits August 18, 2023 11:23
feat: add dataclass to model unite legale response
416abef
feat: add dataclass to model unite legale response
a3ae8c6
chore: update formatter with new response dataclasses
00f5e75
refactor: use ResponseBuilder class to build api response
a5f36ef
feat: format es search results using UniteLegaleResponse dataclass
b6c0224
feat: use pydantic classes to model unite legale
3b25c48
refactor: capture erros in sentry in http exception decorator
861acf8
refactor: add new pydantic classes to field formatters
5fe8f9a
refator: move nom_complet formatter to formatters folder
c15ee1e
refactor: add helps to select fields to include in response
acdd342
chore: add pydantic to requirements
13017b2
refactor: create response model class
ccc5d4a
refactor: use new unite legale class to format search results
962ca68
refactor: add response builder class
a3b8d64
refactor: clean api response builder function
a94049e
refactor: rename route function calls
07abdf0
refactor: add http handler wrapper to api response
f863d5d
refactor: add matching_etablissements to optional fields
c2e84e5
fix: add list to etablissement type
38ffee7
fix: make collectivite territoriale fields optional
daf4d9a
refactor: group non diffusibilite logic in one function call
c9aafcd
feat(packages): add pydantic version
7e74b4c
@HAEKADI HAEKADI marked this pull request as ready for review August 31, 2023 19:35
@HAEKADI HAEKADI self-assigned this Sep 1, 2023
@HAEKADI HAEKADI added documentation Improvements or additions to documentation enhancement New feature or request new filters new API filters new data improve search API payload swagger labels Sep 1, 2023
@XavierJp XavierJp marked this pull request as draft September 14, 2023 08:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@XavierJp XavierJp Awaiting requested review from XavierJp XavierJp will be requested when the pull request is marked ready for review XavierJp is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@HAEKADI HAEKADI

Labels
API payload documentation Improvements or additions to documentation enhancement New feature or request improve search new data new filters new API filters swagger
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@HAEKADI