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

[#19] [Integrate] As a user, I can see Survey selection page #102

Merged
merged 6 commits into from
Jan 13, 2023

Conversation

blyscuit
Copy link
Owner

@blyscuit blyscuit commented Dec 28, 2022

What happened

Describe the big picture of your changes here to communicate to the team why we should accept this pull request.

Insight

Describe in details how to test the changes; referenced documentation is welcome as well.

Proof Of Work

Screen.Recording.2022-12-28.at.16.09.36.mp4

@github-actions
Copy link

github-actions bot commented Dec 28, 2022

5 Warnings
⚠️ Uh oh! SurveyUiModel.kt is under 95% coverage!
⚠️ shared/src/commonMain/kotlin/co/nimblehq/blisskmmic/presentation/modules/surveyselection/SurveySelectionViewModel.kt#L31 - The constructor(getCurrentDateUseCase: GetCurrentDateUseCase, getProfileUseCase: GetProfileUseCase, getAppVersionUseCase: GetAppVersionUseCase, surveyListUseCase: SurveyListUseCase, dateTimeFormatter: DateTimeFormatter) has too many parameters. The current threshold is set to 5.
⚠️ shared/src/commonMain/kotlin/co/nimblehq/blisskmmic/presentation/modules/surveyselection/SurveySelectionViewModel.kt#L31 - Class 'SurveySelectionViewModel' with '13' functions detected. Defined threshold inside classes is set to '10'
⚠️ shared/src/commonTest/kotlin/co/nimblehq/blisskmmic/data/repository/AuthenticationRepositoryTest.kt#L23 - Class 'AuthenticationRepositoryTest' with '10' functions detected. Defined threshold inside classes is set to '10'
⚠️ shared/src/commonTest/kotlin/co/nimblehq/blisskmmic/presentation/modules/surveyselection/SurveySelectionViewModelTest.kt#L34 - Class SurveySelectionViewModelTest is too large. Consider splitting it into smaller pieces.

🧛 shared Code Coverage: 83.51%

Coverage of Modified Files:

File Coverage
SurveySelectionViewModel.kt 96.77%
SurveyUiModel.kt 78.05%

Modified Files Not Found In Coverage Report:

FadePaginationView.swift
LoginScreen.swift
Shared+AutoMockable.swift
Survey Staging Extended Tests.xcscheme
SurveyScreen.swift
SurveySelectionSpec.swift
SurveySelectionView+DataSource.swift
SurveySelectionView.swift
SurveySelectionViewDataSourceSpec.swift
SurveySelectionViewModelTest.kt
project.pbxproj

Codebase cunningly covered by count Shroud 🧛

Generated by 🚫 Danger

@github-actions
Copy link

github-actions bot commented Dec 28, 2022

Current coverage for Survey is 88.68%

Files changed - -
SurveySelectionView+DataSource.swift 88.89%
FadePaginationView.swift 95.78%
SurveySelectionView.swift 98.68%

Powered by xcov

Generated by 🚫 Danger

@blyscuit blyscuit force-pushed the feature/#19-survey-selection-integration branch 2 times, most recently from d67c5d3 to a966163 Compare December 29, 2022 02:10
@blyscuit blyscuit added this to the 0.5.0 milestone Dec 29, 2022
@blyscuit blyscuit self-assigned this Dec 29, 2022
@blyscuit blyscuit changed the title [#19] Integrate survey selection [#19] [iOS] [Integrate] As a user, I can see Survey selection page Dec 29, 2022
@blyscuit blyscuit force-pushed the feature/#24-logout-integration branch 2 times, most recently from dda8d54 to 448cb73 Compare January 3, 2023 08:29
@blyscuit blyscuit force-pushed the feature/#19-survey-selection-integration branch from a966163 to 1809a3e Compare January 3, 2023 08:42
@blyscuit blyscuit force-pushed the feature/#24-logout-integration branch 3 times, most recently from 18e67c9 to fc60eba Compare January 4, 2023 11:38
@blyscuit blyscuit force-pushed the feature/#19-survey-selection-integration branch from 1809a3e to 7e8640e Compare January 5, 2023 09:11
@blyscuit blyscuit force-pushed the feature/#24-logout-integration branch from 48cbf91 to 9adb660 Compare January 5, 2023 09:12
@blyscuit blyscuit force-pushed the feature/#19-survey-selection-integration branch 2 times, most recently from 54db60f to 2fc5af9 Compare January 5, 2023 09:50
@blyscuit blyscuit marked this pull request as ready for review January 5, 2023 09:58
Base automatically changed from feature/#24-logout-integration to develop January 10, 2023 03:25
@blyscuit blyscuit changed the title [#19] [iOS] [Integrate] As a user, I can see Survey selection page [#19] [Integrate] As a user, I can see Survey selection page Jan 10, 2023
Copy link
Collaborator

@doannimble doannimble left a comment

Choose a reason for hiding this comment

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

A few tiny concerns, rest lgtm

@blyscuit blyscuit force-pushed the feature/#19-survey-selection-integration branch from 2fc5af9 to d664cdc Compare January 10, 2023 11:04
Copy link
Collaborator

@minhnimble minhnimble left a comment

Choose a reason for hiding this comment

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

Please take care of the CI failed issue 🛠, implementation-wise no more concern from me 🚀

@blyscuit blyscuit merged commit e021c3e into develop Jan 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants