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/3 fetch images #8

Merged
merged 3 commits into from
Jan 15, 2025
Merged

Feature/3 fetch images #8

merged 3 commits into from
Jan 15, 2025

Conversation

yujung5873
Copy link
Collaborator

#️⃣연관된 이슈

#3

📝작업 내용

  1. 검색창에 적은 텍스트를 오른쪽 보내기 버튼을 누르면 해당 단어의 리스트를 받아오도록 수정
  2. mvvm 패턴과 코루틴으로 view 분리
스크린샷 2025-01-12 오전 12 36 59

###TODO

  1. notifyDataSetChange() 변경 필요

@yujung5873 yujung5873 requested a review from f-lab-tony January 11, 2025 15:37
@yujung5873 yujung5873 self-assigned this Jan 11, 2025
1. 검색창에 적은 텍스트를 오른쪽 보내기 버튼을 누르면 해당 단어의 리스트를 받아오도록 수정
2. mvvm 패턴과 코루틴으로 view 분리

+ main -> feature3 rebase

TODO
1. notifyDataSetChange() 변경 필요
@yujung5873 yujung5873 force-pushed the feature/3-fetch-images branch from c34d188 to 132ed55 Compare January 15, 2025 11:28
initSearchView() 에서 불필요한 CoroutineScope 사용 제거
예외 상황에서는 null을 반환하고, errorToast를 띄우도록 수정
-> 성공했지만 검색 결과가 empty인 경우와 구분
@yujung5873 yujung5873 merged commit fa0c96c into main Jan 15, 2025
2 checks passed
@yujung5873 yujung5873 deleted the feature/3-fetch-images branch January 15, 2025 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants