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

Feature #164 feature model 계층으로 리팩터링 #165

Merged
merged 10 commits into from
Nov 27, 2024
Merged

Conversation

ATeals
Copy link
Collaborator

@ATeals ATeals commented Nov 26, 2024

#️⃣연관된 이슈

#164

📝작업 내용

  • 코드베이스에 흩어진 도메인에 대한 타입을 model class를 통해 응집
  • 도메인의 비즈니스 로직을 model class에 선언
  • api dto를 통해 도메인 model 변환 후 ui에서 사용

💬리뷰 요구사항(선택)

[번역] 잘 알려진 UI 패턴을 사용하여 리액트 애플리케이션 모듈화하기

코드베이스의 전반적인 타입을 맞추다 보니... 변경사항이 매우 많습니다. 커밋을 feature 단위로 끊어서 올렸습니다. 🙇

kimhji and others added 4 commits November 27, 2024 00:15
* feat(#117): lotus 목록 페이지 error fallback 처리하기

* feat(#117): 사용자 정보 페이지 error fallback 처리하기

* feat(#117): lotus 상세 페이지 error fallback 처리하기

* feat(#117): lotus 생성 페이지 error fallback 처리하기

* refactor(#117): errorFallback -> Error 컴포넌트로 이름 수정하기

* refactor(#117): 페이지 전역 에러 처리 레이아웃을 GlobalError 컴포넌트로 분리

* fix(#117): gistFile 뷰에서 에러 발생 시에도 높이, 너비 유지하기
* feat(#117): lotus 목록 페이지 error fallback 처리하기

* feat(#117): 사용자 정보 페이지 error fallback 처리하기

* feat(#117): lotus 상세 페이지 error fallback 처리하기

* feat(#117): lotus 생성 페이지 error fallback 처리하기

* refactor(#117): errorFallback -> Error 컴포넌트로 이름 수정하기

* refactor(#117): 페이지 전역 에러 처리 레이아웃을 GlobalError 컴포넌트로 분리

* fix(#117): gistFile 뷰에서 에러 발생 시에도 높이, 너비 유지하기
Copy link
Collaborator

@naarang naarang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

와ㅠㅠ 수고 많으셨습니다!!🙇‍♀️

@ATeals ATeals merged commit 762c10b into main Nov 27, 2024
2 checks passed
@ATeals ATeals deleted the feature-#164 branch November 27, 2024 01:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants