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.

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

shared_kernel 도메인 추가 #7

Merged
merged 2 commits into from
Feb 1, 2021
Merged

shared_kernel 도메인 추가 #7

merged 2 commits into from
Feb 1, 2021

Conversation

heumsi
Copy link
Collaborator

@heumsi heumsi commented Feb 1, 2021

🧑‍💻 PR 내용

OutputDto 통일을 위한 shared_kernel 추가

📸 스크린샷

스크린샷을 첨부해주세요.

@heumsi heumsi merged commit c90d6ba into develop Feb 1, 2021
@heumsi heumsi deleted the feature/shared_kernel branch February 1, 2021 14:57
heumsi added a commit that referenced this pull request Feb 27, 2021
* docs: PR 규칙의 제목 부분 수정

* 유저 기본 기능 관련 PR (#3)

* build: dependency-injector 라이브러리 추가

* feat: domain 레이어 추가

* feat: application 레이어 추가

* feat: infra structure 레이어 추가

* feat: external interface 및 main.py 추가

* refactor: tests/user 를 tests/users 로 변경

* refactor: test_routers 에서 사용하지 않는 코드 삭제

* build: requirements.txt 추가 (from poetry export)

* refactor: User 엔티티에서 reivews_id, wished_drink_ids 삭제

* test: 애플리케이션 서비스 테스트 코드에서 누락된 부분 추가

* health check 용 도메인 별도로 분리 (#2)

* build: Dockerfile 추가

* shared_kernel 도메인 추가 (#7)

* feat: shared_kernel 도메인 추가

* fix: status, message 파라미터 이름 추가

* Shared kernel 도메인에 SuccessOutputDto 및 __bool__ 메서드 추가 수정 (#8)

* refactor: SuccessOutputDto 및 __bool__ 메서드 추가

* refactor: SuccessOutputDto 에 status 삭제

* refactor: FailedOutputDto 에서 status -> type 으로 이름 변경

* feat: Shared kernel에 FailedJsonResponse 추가 (#9)

* feat: shared_kernel 에서 추가된 부분에 따라 service, routers 모두 수정 (#10)

* refactor: OutputDto 의 상태를 bool이 아닌 status 로 표현 (#11)

* feature: Auth 관련 기능 PR (#12)

* feat: auth 도메인 추가

* fix: UserApplicationService.login 에 있던 버그 수정

* fix: containers 이동에 따른 import 구문 변경

* fix: shared_kernel 에 있던 잘못된 테스트 수정

* refactor: users routers 에서 UserApplicationService 를 주입받도록 수정

* feat: 리뷰 기능 관련 PR (#5)

* feat: domain layer 추가

* feat: application layer 추가

* feat: infra_structure layer 추가

* feat: external interface layer 추가

* test: modify reviews tests

* refactor: Add Dockerfile and reviews init file

* black 으로 포매팅

Co-authored-by: heumsi <[email protected]>

* build: poetry.lock 및 requirements.txt 업데이트

* feat: Auth 에 verfiy token 기능 추가 (#14)

* feat: verify_token 기능 추가

* build: commitzen, semantic-release 라이브러리 추가

* build: isort 라이브러리 추가

* style: black, isort 전체적으로 적용

* feat: Reviews 관련 수정 PR (#15)

* test: Reviews application service 관련 테스트 추가

* feat: domain layer 추가

* feat: application layer 추가

* feat: infra_structure layer 추가

* feat: external interface layer 추가

* style: black, isort 적용

Co-authored-by: heumsi <[email protected]>

* feat: auth 에 토큰 데이터 가져오는 기능 추가 (#16)

* refactor: Reviews 서비스 수정 (#17)

* feat: application layer 추가

* feat: infra_structure layer 추가

* feat: external interface layer 추가

* style: code reformatting

* feat: Users 에 CRUD 전체 및 패스워드 해시 기능 추가 (#18)

* feat: crud 에서 나머지 기능 (rud) 구현

* feat: password 를 hashed 하는 기능 추가

* test: 테스트 코드 추가

* feat: Reviews 에 CRUD 관련 라우터 추가 및 리팩토링 (#19)

* style: import 코드 재정렬(Optimize)

* refactor: 사용하지 않는 exceptions.py 삭제

* feat: CRUD 관련 엔드포인트 추가 (GET /reviews/ 만 제외) 및 코드 전반 리팩토링

* feat: wish 도메인 추가 (#20)

* feat: drinks 도메인 추가

* test: Drinks service 관련 tests 추가

* feat: domain layer 추가

* feat: application layer 추가

* feat: infra_structure layer 추가

* feat: external interface layer 추가

* feat: VO 에 filtering Enums 추가

* feat: drink review add, delete methods 추가

* test: find by drink id, create, update drink review (add, delete) 테스트

* feat: Application service 에 update, delete methods 추가

* feat: Drinks find_all with filtering 기능 추가

* feat: External interface에 POST, GET 추가

* test: drinks router tests 추가, application service test 완료

* feat: wishes service, drinks service 에 wish 추가, 삭제 기능 추가

* refactor: DrinkId VO 추가

* test: wishes test_service에 drinks 관련 기능 추가

* feat: review Create, Update, Delete시 drinks rating update기능 추가

* refactor: drinks router QueryParam 추가

* refactor: import optimization

* style: formatting with black and isort

* refactor: Users 의 Repository 를 ORM 으로 바꾸는 작업 (#21)

* refactor: Users 의 Repository 를 SQLAlchemy 형태(ORM)로 수정

* feat: orm repository 반영하여 users 수정

* test: test code 추가

* refactor: shared kernel 수정

* refactor: Wish 의 Repository 를 ORM 으로 바꾸는 작업  (#22)

* format: Optimize import

* refactor: DrinkId 를 shared_kernel 로 이동

* feat: WishId VO 추가

* feat: ORM Repository 추가

* refactor: delete_by_wish_id 의 리턴 값을 None -> Wish 로 수정

* refactor: 변경된 Repository 에 맞춰 Service 도 수정

* refactor: container.py 에서 InMemory -> ORM Repository 로 교체

* refactor: routers 쪽 모두 수정

* refactor: users, wishes 의 ORM Model Column 타입 변경 (#23)

* refactor: postgres 로 테스트용 디비 수정 및 ORM Model 을 UUID 타입으로 수정 (#25)

* build: psycopg2-binary 라이브러리 설치

* refactor: postgres 에 맞게 테스트 DB_URL 도 변경

* refactor: UUID 를 이용하여 ORM 컬럼 타입 구성

* refactor: users 에서 sqlalchemy session 을 이용하여 테스트 데이터 삽입하도록 수정 (#26)

* test: auth 에서 망가진 test code 다시 작성

* refactor: reviews, drinks ORM 및 테스트 수정

* test: 환경변수에 TEST_DB_URL 추가

* test: reviews tests update

* build: dev 에 coverage 추가

* fix: auth routers get_token 쪽에 response_model 빠진거 추가

* test: .coveragerc 추가

* fix: auth routers get_token 의 요청 메서드를 GET -> POST 로 수정

* refactor: create drinks drink_id제거, to_enum function 추가

* fix: GET /drinks 요청 시 response 잘못 나오는거 수정

* test: auth, get token GET -> POST방식으로 수정

* style: import 최적화, black 포매팅

* refactor: json_dto -> json_dtos 로 모듈 이름 변경

* docs: README.md 업데이트

* fix: json_dtos 로 바꾸고 나서 import 쪽에 수정안된 부분 픽스

* build: coverage-badge 라이브러리 추가

* docs: coverage badge 추가

* docs: changelog.md 추가

* 0.1.0

Automatically generated by python-semantic-release

Co-authored-by: Seok Jun Hong <[email protected]>
Co-authored-by: mayjunjuly0313 <[email protected]>
Co-authored-by: semantic-release <semantic-release>
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