-
Notifications
You must be signed in to change notification settings - Fork 0
Conversation
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.
nice work, LGTM 👍 two minor things below
app/src/androidTest/java/ch/sdp/vibester/fragment/SearchUserFragmentNotLoggedInTest.kt
Outdated
Show resolved
Hide resolved
app/src/androidTest/java/ch/sdp/vibester/fragment/SearchUserFragmentNotLoggedInTest.kt
Outdated
Show resolved
Hide resolved
…agmentNotLoggedInTest.kt Co-authored-by: Maxime <[email protected]>
…agmentNotLoggedInTest.kt Co-authored-by: Maxime <[email protected]>
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.
👍
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.
Loooks good! I have changed the string, let me know if you agree!
Co-authored-by: kamilababayeva <[email protected]>
Code Climate has analyzed commit 8711916 and detected 0 issues on this pull request. The test coverage on the diff in this pull request is 100.0% (80% is the threshold). This pull request will bring the total coverage in the repository to 80.6% (0.0% change). View more on Code Climate. |
No description provided.