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: PokitFeature 액션 수정 #147

Merged
merged 1 commit into from
Oct 20, 2024
Merged

Conversation

stealmh
Copy link
Contributor

@stealmh stealmh commented Oct 18, 2024

#️⃣연관된 이슈

#129

📝작업 내용

  • 액션 컨벤션에 맞춰 수정
  • 파라미터로 직접넣는 방법에서 미리 선언해 이를 넣어주는 방식으로 일부 액션 수정

스크린샷 (선택)

💬리뷰 요구사항(선택)

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

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

@stealmh stealmh requested a review from ShapeKim98 October 18, 2024 04:56
@stealmh stealmh self-assigned this Oct 18, 2024
@stealmh stealmh added the Refactor 🏗️ 뚝딱뚝딱 코드 및 구조 수정 label Oct 18, 2024
@stealmh stealmh linked an issue Oct 18, 2024 that may be closed by this pull request
14 tasks
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.

수고하셨습니다!

@@ -256,40 +252,40 @@ private extension PokitRootFeature {
/// - Inner Effect
func handleInnerAction(_ action: Action.InnerAction, state: inout State) -> Effect<Action> {
switch action {
case let .pokitCategorySheetPresented(presented):
case let .카테고리_시트_활성화(presented):
Copy link
Contributor

Choose a reason for hiding this comment

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

앞으로 뷰 띄우는 액션은 활성화라고 하면 될까여?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

넵 ~~_활성화(true), _활성화(false)가 더 눈에 잘 들어오는것 같아요

@stealmh stealmh merged commit 38c2bb7 into develop Oct 20, 2024
1 check passed
@stealmh stealmh deleted the refactor/#129-PokitFeature branch November 21, 2024 14:47
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