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

[feat] 번개 참여 및 취소하기 API 연동 #105

Merged
merged 15 commits into from
Jan 11, 2024

Conversation

jihyunniiii
Copy link
Collaborator

Related issue 🛠

Work Description ✏️

  • 번개 참여 및 취소하기 API를 연동했습니다.
  • 자동로그인 시 앱 터지는 문제를 해결했습니다.

Screenshot 📸

Screen_recording_20240111_205649.mp4

Uncompleted Tasks 😅

  • N/A

To Reviewers 📢

ㅋㅋ 다했당

Copy link
Collaborator

@HAJIEUN02 HAJIEUN02 left a comment

Choose a reason for hiding this comment

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

짱이다!! 고생했핑!!

Copy link
Collaborator

@HAJIEUN02 HAJIEUN02 left a comment

Choose a reason for hiding this comment

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

굿굿핑 잘했핑 식상하네요

override suspend fun postPingleJoin(meetingId: Long): NullableBaseResponse<Unit?> =
pingleService.postPingleJoin(meetingId = meetingId)

override suspend fun postPingleCancel(meetingId: Long): NullableBaseResponse<Unit?> =
Copy link
Collaborator

Choose a reason for hiding this comment

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

없는 경우에는 그냥 BaseResponse안 쓰고 Call하면 된다고 생각했는데 Call은 enqueue에만 쓰이는 개념이라는 걸 오늘 알게 되었습니다.. 대박

AllModalDialogFragment(
title = stringOf(R.string.map_cancel_modal_title),
detail = stringOf(R.string.map_cancel_modal_detail),
buttonText = stringOf(R.string.map_cancel_modal_button_text),
textButtonText = stringOf(R.string.map_cancel_modal_text_button_text),
clickBtn = { },
clickTextBtn = { },
onDialogClosed = { }
Copy link
Collaborator

Choose a reason for hiding this comment

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

이건 왜 사라진 건가염?

@@ -33,9 +37,12 @@ class MapViewModel @Inject constructor(
private var _selectedMarkerPosition = MutableStateFlow(DEFAULT_SELECTED_MARKER_POSITION)
val selectedMarkerPosition get() = _selectedMarkerPosition.asStateFlow()

private val _pingleListState = MutableSharedFlow<UiState<List<PingleEntity>>>()
private val _pingleListState = MutableSharedFlow<UiState<Pair<Long, List<PingleEntity>>>>()
Copy link
Collaborator

Choose a reason for hiding this comment

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

허걱!!! Long과 PingleEntity List 두가지를 한꺼번에 return하기 위해 사용한 건가욤??!?!? 야무야무

@@ -165,6 +165,8 @@
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:textAppearance="@style/TextAppearance.Pingle.Sub.Semi.16"
android:layout_marginEnd="2dp
"
Copy link
Collaborator

Choose a reason for hiding this comment

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

머죠 이게 ㅋㅋ

# Conflicts:
#	app/src/main/java/org/sopt/pingle/di/DataSourceModule.kt
#	app/src/main/java/org/sopt/pingle/di/RepositoryModule.kt
#	app/src/main/java/org/sopt/pingle/di/ServiceModule.kt
@jihyunniiii jihyunniiii merged commit bd27bfb into develop Jan 11, 2024
1 check passed
@jihyunniiii jihyunniiii deleted the feat-pingle-api-connection branch January 11, 2024 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[feat] 번개 참여 및 취소하기 API 연동
2 participants