-
Notifications
You must be signed in to change notification settings - Fork 0
Feat/#5 mypage screen구현 #37
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
The head ref may contain hidden characters: "feat/#5-mypage-screen\uAD6C\uD604"
Changes from all commits
b548355
3c920ce
9f1666e
b422e57
5fe71c7
1ba0317
4c48297
6baf46a
195c8b8
389f96d
51e455a
1e14e81
6d73778
911429e
642c5bb
c7a856b
48942a9
14ae354
87eddce
24ea079
e9b3f9f
00ad875
81db329
634b84c
694bdd4
7cfa612
e2c63bf
2a967df
95b71b1
a461e2a
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,10 +1,8 @@ | ||
| package com.umcspot.spot.weather.dto.response | ||
|
|
||
| import android.annotation.SuppressLint | ||
| import com.umcspot.spot.model.WeatherType | ||
| import kotlinx.serialization.SerialName | ||
| import kotlinx.serialization.Serializable | ||
| import java.time.LocalTime | ||
|
|
||
| @SuppressLint("UnsafeOptInUsageError") | ||
| @Serializable | ||
|
|
@@ -20,7 +18,7 @@ data class WeatherResponse( | |
| val header: WeatherHeader, | ||
|
|
||
| @SerialName("body") | ||
| val body: WeatherBody | ||
| val body: WeatherBody? = null | ||
|
Comment on lines
20
to
+21
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧩 Analysis chain🏁 Script executed: #!/bin/bash
# Search for all usages of response.body to check null handling
rg -n --type=kotlin 'response\.body' -C3Repository: SPOTeam/Android-V2 Length of output: 814 🏁 Script executed: #!/bin/bash
# Search for all usages of toDomain() to see error handling context
rg -n --type=kotlin 'toDomain\(\)' -C3
# Also search for WeatherResponseDto usages
rg -n --type=kotlin 'WeatherResponseDto' -C2
# Check if body is accessed in other ways
rg -n --type=kotlin '\.body' data/weather -C2Repository: SPOTeam/Android-V2 Length of output: 24024 🏁 Script executed: #!/bin/bash
# Check the full context around the toDomain() call in WeatherRepositoryImpl
sed -n '60,85p' data/weather/src/main/java/com/umcspot/spot/weather/repositoryimpl/WeatherRepositoryImpl.ktRepository: SPOTeam/Android-V2 Length of output: 854
현재는 WeatherMapper.kt 라인 10val items = response.body!!.items.item🤖 Prompt for AI Agents |
||
| ) | ||
|
|
||
| @SuppressLint("UnsafeOptInUsageError") | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
복사-붙여넣기 오류: 잘못된 로그 메시지.
getWaitingStudy메서드에서 로그 메시지가"getRecruitingStudy failed"로 잘못 표시되어 있습니다. 디버깅 시 혼란을 줄 수 있습니다.🐛 제안하는 수정
}.onFailure { e -> - Log.e("StudyRepository", "getRecruitingStudy failed", e) + Log.e("StudyRepository", "getWaitingStudy failed", e) }📝 Committable suggestion
🤖 Prompt for AI Agents