Skip to content
This repository has been archived by the owner on Dec 31, 2023. It is now read-only.

[1.0.0] 최근 검색어를 저장하는 로직을 추가 #3

Closed
wants to merge 24 commits into from

Conversation

lgvv
Copy link
Member

@lgvv lgvv commented Jun 8, 2022

[1.0.0] 최근 검색어를 저장하는 로직을 추가

  • 여기 PR은 fork한 레포에서 작업을 진행하였습니다.

검색어는 최근 검색어가 제일 앞쪽에 위치합니다.
이미 존재하는 검색어는 자동으로 사라지고, 좌우로 스크롤이 가능하게 만들었습니다.
또한 검색어 선택시, 자동으로 선택하여 검색을 수행합니다.
검색어 옆에 X버튼을 클릭하면 최근 검색어에서 사라집니다.

@x-0o0 x-0o0 self-requested a review June 8, 2022 18:00
@x-0o0 x-0o0 changed the base branch from develop/lgvv/swiftui-recent-search to main June 8, 2022 18:16
Copy link
Contributor

@x-0o0 x-0o0 left a comment

Choose a reason for hiding this comment

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

1차 리뷰를 완료하였습니다. 로직관련 수정이 필요해보여서 2차 리뷰도 진행하겠습니다.

KuringLite/Manager/AppStorageManager.swift Outdated Show resolved Hide resolved
KuringLite/Search/DataModel/SearchEngine.swift Outdated Show resolved Hide resolved
KuringLite/Search/DataModel/SearchEngine.swift Outdated Show resolved Hide resolved
KuringLite/Search/View/SearchView.swift Outdated Show resolved Hide resolved
KuringLite/Search/View/SearchView.swift Show resolved Hide resolved
KuringLite/Search/View/SearchView.swift Outdated Show resolved Hide resolved
KuringLite/Search/View/SearchedRecentView.swift Outdated Show resolved Hide resolved
Copy link
Contributor

@x-0o0 x-0o0 left a comment

Choose a reason for hiding this comment

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

2차리뷰 완료했습니다. 데이터모델 관련 로직에 대한 리뷰를 했습니다.

KuringLite/Search/DataModel/SearchEngine.swift Outdated Show resolved Hide resolved
KuringLite/Search/DataModel/SearchEngine.swift Outdated Show resolved Hide resolved
KuringLite/Search/View/SearchView.swift Outdated Show resolved Hide resolved
KuringLite/Search/View/SearchedRecentView.swift Outdated Show resolved Hide resolved
@lgvv
Copy link
Member Author

lgvv commented Jun 13, 2022

UI를 개선해보려고 합니다.
[기능구현은 마무리 되었습니다]
참고 - 배달의 민족, 당근마켓을 포함하여 다양한 리소스
UI개선 후 wip를 제거하여 코드 리뷰를 요청하도록 하겠습니다.

@lgvv lgvv added 디자인 리뷰 기능 구현은 마쳤으며, 디자인 승인 대기중 and removed work in progress labels Jun 17, 2022
@x-0o0
Copy link
Contributor

x-0o0 commented Sep 5, 2022

  1. 팀에 디자인 리뷰 요청 넣어주세요~
  2. main과의 conflict을 수정했습니다. 로컬 브랜치 pull 한번 해주시고 기능 잘 돌아가는지 확인 부탁드려요

@x-0o0 x-0o0 assigned x-0o0 and lgvv and unassigned x-0o0 Sep 5, 2022
@lgvv
Copy link
Member Author

lgvv commented Sep 8, 2022

  1. 팀에 디자인 리뷰 요청 넣어주세요~
    감사합니다!! 이것도 추석 연휴 기간에 한번 말씀드려 보겠습니다.
  2. main과의 conflict을 수정했습니다. 로컬 브랜치 pull 한번 해주시고 기능 잘 돌아가는지 확인 부탁드려요
    이 부분도 감사합니다!! 제가 꼼꼼하게 체크하도록 하겠습니다!
  • 추가로 제가 이미지로 올려 드렸는데, GIF로도 올릴 수가 있더라고요!!
    다음부터는 GIF로 바로 확인할 수 있도록 올려드리도록 하겠습니다!!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
디자인 리뷰 기능 구현은 마쳤으며, 디자인 승인 대기중
Projects
None yet
2 participants