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

Refactoring/post search #177

Merged
merged 22 commits into from
Apr 23, 2024
Merged

Refactoring/post search #177

merged 22 commits into from
Apr 23, 2024

Conversation

bdrsky2010
Copy link
Contributor

개요 (이슈 번호 및 요약)

변경 사항 (작업 내용)

스크린샷

특이사항 (트러블 슈팅 과정 및 참고 자료 등)

📕 게시글 실시간 처리

PostViewModel.swift

  • 실시간 리스너
    Pasted image 20240421181247

  • 실시간 리스너 구독
    Pasted image 20240421181609

    • 해당 메서드 호출 (PostView.swift)
      Pasted image 20240421182411
  • 실시간 리스너 구독 취소 (아직 해당 메서드 호출 타이밍을 모르겠음)
    Pasted image 20240421181633

📗 게시글 검색

  • 먼저 검색하면 파이어베이스에서 데이터를 받아와서 타입별로 그에 해당하는 개수만 파악

    • PostViewModel.swift 수정

      • 속성 추가
        Pasted image 20240420184340

      • 수정으로 인한 속성 제거
        Pasted image 20240421175952

        Pasted image 20240421180022

      • 메서드 추가

        • 라인 181 ~ 238
      • 메서드 수정

        • 라인 240 ~ 288
      • 메서드 삭제
        Pasted image 20240421180918

    • PostSearchList.swift 수정

      • 카운트용 속성이 추가 됨에 따라 전달인자 변경
        Pasted image 20240420184936

        Pasted image 20240420185056

  • NavigationPostView.swift 수정

    • 검색 리스트뷰 혹은 PostDetailView를 통해 들어가게되면

    • 데이터 패치 후 정렬
      Pasted image 20240420214004

    • 검색, 음식 태그 이동 둘의 통합으로 인한 코드 수정
      Pasted image 20240421174541

    • 정렬방식 변경 시
      Pasted image 20240420214301

    • enum case 통합으로 인한 조건문 수정
      Pasted image 20240421201143

    • 로직 수정에 따른 불필요한 속성 제거
      Pasted image 20240421174725
      Pasted image 20240421175330

  • NavigationPostView.swift 속성 제거로 인한 NavigationRouter.swift enum 수정
    Pasted image 20240421180502
    Pasted image 20240421180613

  • PostTags.swift 수정

    • NavigationStack Route enum PostSearch랑 통합
      Pasted image 20240421174025
      Pasted image 20240421174109

일단 배열보내주고 테스트
테스트 아직 미완료
상세 로직은 PR 혹은 Issue에 설명하겠음
메서드 명은 임시로 해놓은거라 다시 수정예정
@bdrsky2010 bdrsky2010 added 버그 무엇인가가 잘못되었다. 기능 새로운 기능 구현 데이터 데이터 관련 작업 수행 수정 코드 리팩토링 수행 labels Apr 21, 2024
@bdrsky2010 bdrsky2010 added this to the PostView milestone Apr 21, 2024
@bdrsky2010 bdrsky2010 requested review from Phangg and withseon April 21, 2024 11:35
@bdrsky2010 bdrsky2010 self-assigned this Apr 21, 2024
@Phangg
Copy link
Member

Phangg commented Apr 21, 2024

굿굿- 고생하셨습니다- 😄

@withseon withseon linked an issue Apr 22, 2024 that may be closed by this pull request
@withseon
Copy link
Contributor

검색 기능이 여기저기 신경 쓸 부분이 많아서 까다로웠을텐데 고생하셨습니다 -! 🥂

@bdrsky2010 bdrsky2010 merged commit 0a90f7f into develop Apr 23, 2024
@bdrsky2010 bdrsky2010 deleted the refactoring/PostSearch branch April 23, 2024 05:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
기능 새로운 기능 구현 데이터 데이터 관련 작업 수행 버그 무엇인가가 잘못되었다. 수정 코드 리팩토링 수행
Projects
None yet
Development

Successfully merging this pull request may close these issues.

PostSearch 관련 에러 고치는중..
3 participants