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

#191/feature/jsp/navigation #192

Merged
merged 7 commits into from
Jul 26, 2023
Merged

Conversation

pknujsp
Copy link
Collaborator

@pknujsp pknujsp commented Jul 26, 2023

πŸ“Œ κ΄€λ ¨ 이슈

#187
#191

✨ 과제 λ‚΄μš©

  • μ•½ 상세 정보 ν™”λ©΄μ˜ νˆ΄λ°” λ ˆμ΄μ•„μ›ƒ κ°œμ„ 
  • μ•½ 상세 정보 응닡 λ‚΄ xmlνŒŒμ‹± ν…ŒμŠ€νŠΈ μ½”λ“œ μΆ”κ°€
  • μ•½ 상세 정보 μ‘°νšŒμ‹œ 응닡 ν–‰ 개수 1개둜 μˆ˜μ •
  • μ•½ 상세 정보 μ‘°νšŒν• λ•Œ String으둜 응닡받아 μ˜¨ν›„ 데이터 객체둜 λ³€ν™˜ν•˜λŠ” λ°©μ‹μœΌλ‘œ 둜직 λ³€κ²½
    • μΊμ‹œλ₯Ό μ €μž₯ν•΄μ•Ό ν•˜λŠ”λ°, μ΄λ•Œ λ¬Έμžμ—΄λ‘œ μ €μž₯ν•˜κΈ° λ•Œλ¬Έμ— μ΄λ ‡κ²Œ λ°”κΏ¨μŠ΅λ‹ˆλ‹€.
  • BottomNav둜 ν™”λ©΄ μ „ν™˜ μ‹œ μΈμŠ€ν„΄μŠ€ 재 생성 ν•˜μ§€ μ•Šλ„λ‘ ν•˜λŠ” μ½”λ“œ μΆ”κ°€ -> 정상 μž‘λ™ν•˜μ§€ μ•Šμ•„ 보λ₯˜
  • λͺ‡ 개 λΌμ΄λΈŒλŸ¬λ¦¬μ— λŒ€ν•΄μ„œ 버전 μ—…λ°μ΄νŠΈ ν–ˆμŠ΅λ‹ˆλ‹€.

πŸ“Έ μŠ€ν¬λ¦°μƒ·(선택)

πŸ“š 레퍼런슀 (λ˜λŠ” μƒˆλ‘œ μ•Œκ²Œ 된 λ‚΄μš©) ν˜Ήμ€ κΆκΈˆν•œ 사항듀

@pknujsp pknujsp added enhancement κΈ°λŠ₯ λ°•μ€€μ„± μ•ˆλ“œλ‘œμ΄λ“œ πŸ”¨refactorπŸ”¨ μ½”λ“œ λ¦¬νŒ©ν„°λ§! πŸ”₯πŸ”₯ μ½”λ“œ 리뷰 κΈ°λ‹€λ¦¬λŠ” 쀑 πŸ”₯πŸ”₯ PR을 올리고 μ½”λ“œλ¦¬λ·°λ₯Ό 기닀리고 μžˆλŠ” μƒνƒœμž…λ‹ˆλ‹€. labels Jul 26, 2023
@pknujsp pknujsp requested a review from tgyuuAn July 26, 2023 05:05
Copy link
Member

Choose a reason for hiding this comment

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

ν…ŒμŠ€νŠΈ μ‹œμž‘ν•˜μ‹ κ±°κ΅°μš”,,,, μ €λŠ” λ°°μš°λŠ” 쀑...

@@ -51,7 +50,7 @@ class HomeFragment :

private fun handleEvent(event: HomeViewModel.HomeEvent) {
when (event) {
is HomeViewModel.HomeEvent.NavigateToSearch -> navigateWithNavDirections(HomeFragmentDirections.actionHomeFragmentToSearchMedicinesFragment())
is HomeViewModel.HomeEvent.NavigateToSearch -> findNavController().navigate(R.id.action_homeFragment_to_searchMedicinesFragment)
Copy link
Member

Choose a reason for hiding this comment

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

findNavController().navigate(R.id.action_homeFragment_to_searchMedicinesFragment)
λ₯Ό ꡳ이
navigateWithNavDirections(HomeFragmentDirections.actionHomeFragmentToSearchMedicinesFragment())
둜 μ•ˆ 싸도 될 것 κ°™μ•„μ„œ λΉΌμ‹ κ±΄κ°€μš” ?!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

κΈ°λŠ₯ λ§Œλ“œλŠ” κ±° ν™•μΈν•˜λ‹€κ°€ κ·ΈλŒ€λ‘œ λ†”λ’€λ„€μš” λ‹€ μ™„μ„±λ˜λ©΄ 맞게 μˆ˜μ •ν• κ²Œμš”

@tgyuuAn tgyuuAn added βš’οΈβš’οΈ 리뷰 μ™„λ£Œ μˆ˜μ • ν•„μš”ν•©λ‹ˆλ‹€. βš’οΈβš’οΈ μ½”λ“œ 리뷰가 μ™„λ£Œ λ˜μ—ˆμ§€λ§Œ μˆ˜μ • μ‚¬ν•­μ΄λ‚˜ λ³€κ²½ 사항이 μžˆλŠ” 경우 μž…λ‹ˆλ‹€. and removed πŸ”₯πŸ”₯ μ½”λ“œ 리뷰 κΈ°λ‹€λ¦¬λŠ” 쀑 πŸ”₯πŸ”₯ PR을 올리고 μ½”λ“œλ¦¬λ·°λ₯Ό 기닀리고 μžˆλŠ” μƒνƒœμž…λ‹ˆλ‹€. labels Jul 26, 2023
@pknujsp pknujsp merged commit 7caf6d8 into android_develop Jul 26, 2023
@tgyuuAn tgyuuAn added 🌟🌟 머지 ν•΄μ£Όμ„Έμš” 🌟🌟 μ½”λ“œ 리뷰가 μ™„λ£Œλœ λ’€ PR을 μ˜¬λ¦°μ‚¬λžŒμ΄ Mergeλ₯Ό ν•˜λ©΄ λ˜λŠ” λ‹¨κ³„μž…λ‹ˆλ‹€. and removed βš’οΈβš’οΈ 리뷰 μ™„λ£Œ μˆ˜μ • ν•„μš”ν•©λ‹ˆλ‹€. βš’οΈβš’οΈ μ½”λ“œ 리뷰가 μ™„λ£Œ λ˜μ—ˆμ§€λ§Œ μˆ˜μ • μ‚¬ν•­μ΄λ‚˜ λ³€κ²½ 사항이 μžˆλŠ” 경우 μž…λ‹ˆλ‹€. labels Jul 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
κΈ°λŠ₯ λ°•μ€€μ„± μ•ˆλ“œλ‘œμ΄λ“œ 🌟🌟 머지 ν•΄μ£Όμ„Έμš” 🌟🌟 μ½”λ“œ 리뷰가 μ™„λ£Œλœ λ’€ PR을 μ˜¬λ¦°μ‚¬λžŒμ΄ Mergeλ₯Ό ν•˜λ©΄ λ˜λŠ” λ‹¨κ³„μž…λ‹ˆλ‹€. πŸ”¨refactorπŸ”¨ μ½”λ“œ λ¦¬νŒ©ν„°λ§!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants