✨ ParticipateExecutor を実装 #116
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Walkthrough変更の概要として、認証関連のコードに新たな機能が追加されました。 Changes
TipsChat with CodeRabbit Bot (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Review Status
Actionable comments generated: 2
Configuration used: CodeRabbit UI
Files selected for processing (5)
- core/data/src/commonMain/kotlin/club/nito/core/data/AuthRepository.kt (2 hunks)
- core/data/src/commonMain/kotlin/club/nito/core/data/DefaultAuthRepository.kt (2 hunks)
- core/domain/src/commonMain/kotlin/club/nito/core/domain/ParticipateUseCase.kt (2 hunks)
- core/network/src/commonMain/kotlin/club/nito/core/network/auth/AuthRemoteDataSource.kt (1 hunks)
- core/network/src/commonMain/kotlin/club/nito/core/network/auth/SupabaseAuthRemoteDataSource.kt (1 hunks)
Additional comments: 5
core/data/src/commonMain/kotlin/club/nito/core/data/DefaultAuthRepository.kt (2)
3-7: 新しいメソッド
currentUserでApiException.SessionNotFoundExceptionを投げるために、ApiExceptionのインポートが正しく追加されています。36-37:
currentUserメソッドはremoteDataSourceからcurrentUserOrNullメソッドを正しく使用し、結果がnullの場合にSessionNotFoundExceptionを投げます。core/domain/src/commonMain/kotlin/club/nito/core/domain/ParticipateUseCase.kt (1)
- 3-10: 新しい機能に必要なインポートが追加され、
ParticipateUseCaseインターフェースの説明コメントが追加されています。これらの変更は適切に見えます。core/network/src/commonMain/kotlin/club/nito/core/network/auth/AuthRemoteDataSource.kt (1)
- 16-19: 新しいメソッド
currentUserOrNullがAuthRemoteDataSourceインターフェースに追加されました。このメソッドは、現在ログインしているユーザーの情報を取得するためのものです。この変更は、ユーザー認証とセッション管理に関連する下流の依存関係に影響を与える可能性があります。このインターフェースを実装するクラスは、新しいメソッドを適切に処理する必要があります。core/network/src/commonMain/kotlin/club/nito/core/network/auth/SupabaseAuthRemoteDataSource.kt (1)
- 61-62: 新しいメソッド
currentUserOrNullは、現在認証されているユーザー情報をnull許容で返すように追加されました。この実装は、ユーザーが認証されていない場合にnullを返すという期待される振る舞いと一致しています。
Comment on lines
16
to
28
| public class ParticipateExecutor( | ||
| private val authRepository: AuthRepository, | ||
| private val participantRepository: ParticipantRepository, | ||
| ) : ParticipateUseCase { | ||
| override suspend fun invoke(scheduleId: String, comment: String): ExecuteResult<Unit> { | ||
| TODO("Not yet implemented") | ||
| override suspend fun invoke(scheduleId: String, comment: String): ExecuteResult<Unit> = runExecuting { | ||
| participantRepository.participate( | ||
| declaration = ParticipantDeclaration( | ||
| scheduleId = scheduleId, | ||
| memberId = authRepository.currentUser().id, | ||
| comment = comment, | ||
| ), | ||
| ) | ||
| } |
There was a problem hiding this comment.
authRepository.currentUser().id の呼び出しは例外を投げる可能性があるため、invoke メソッド内での例外処理を検討する必要があります。PRの概要によると currentUser() は例外を投げる可能性があるため、これらの例外を適切に処理することが重要です。
Comment on lines
+33
to
+37
| /** | ||
| * 現在ログイン中のユーザー情報を取得する | ||
| */ | ||
| @Throws(ApiException.SessionNotFoundException::class, CancellationException::class) | ||
| public suspend fun currentUser(): UserInfo |
There was a problem hiding this comment.
新しい関数currentUserがAuthRepositoryインターフェースに追加されましたが、この関数の動作や使用方法についてのドキュメントコメントがありません。公開関数には特に例外を投げる可能性がある場合、その動作を説明するドキュメントが必要です。ドキュメントコメントの追加を検討してください。
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue
概要
ParticipateExecutor を実装します。
レビュー観点
特になし
レビューレベル
レビュー優先度
参考リンク
スクリーンショット
Summary by CodeRabbit
新機能
バグ修正
ドキュメント