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

refactor: SettingFeature 액션 컨벤션 수정 #131

Closed
wants to merge 29 commits into from

Conversation

stealmh
Copy link
Member

@stealmh stealmh commented Sep 29, 2024

#️⃣연관된 이슈

ex) #129

📝작업 내용

  • SettingFeature 모듈 내 Reducer 액션의 이름을 일부 제외하고 한글로 바꾸는 작업 진행

스크린샷 (선택)

💬리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

ShapeKim98 and others added 27 commits September 23, 2024 15:22
- 링크 수정에서 카테고리를 추가하고 돌아오면 추가한 카테고리 명이 적용되지 않던 버그를 수정
- view -> async-> inner로 흐름이 명확하도록 수정
- 영문 네이밍이 익숙한 경우를 제외한 나머지 액션 열거형 한글화
@stealmh stealmh added the Refactor 🏗️ 뚝딱뚝딱 코드 및 구조 수정 label Sep 29, 2024
@stealmh stealmh self-assigned this Sep 29, 2024
@stealmh stealmh requested a review from ShapeKim98 September 29, 2024 08:32
Copy link
Contributor

@ShapeKim98 ShapeKim98 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다! 다음 작업도 이거보고 열심히 반영해보겠습니다!

@stealmh
Copy link
Member Author

stealmh commented Sep 29, 2024

브랜치 실수로 인해 삭제후 PR 올림

@stealmh stealmh closed this Sep 29, 2024
@stealmh stealmh deleted the refactor/#129-SettingFeature branch September 29, 2024 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Refactor 🏗️ 뚝딱뚝딱 코드 및 구조 수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants