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

[Simprints] Bring last changes 3.1.0.1 #248

Conversation

xurxodev
Copy link

@xurxodev xurxodev commented Mar 21, 2025

📌 References

⚙️ branches

app:
Origin: feature-simprints/bring_last_changes_3_1_0_1 Target: develop-spip
dhis2-android-SDK:
Origin: 16080ff19b53f9487d9b44d8dbbabf81c40934ea

🎩 What is the goal?

Bring the last changes in 3.1.0.1

📝 How is it being implemented?

  • Merge conflicts
  • Reimplement or fix custom features
    • Bring cursor bug fix
    • Fix crash to enter in TEI dashboard
Fix error to show verify by hidden crash to parse date (ageInMonths)
    • Hide TEI form in landscape and to add again editButton
    • Remove old compose components to render biometrics in event form
    • Remove unnecesary condition
    • Fix bug calculating months when value is in different formats
    • Fix crash to try show duplicates dialog from TEI form
    • Fix bug to register biometrics from TEI form * Remove unused old code
    • Fix crash:
      java.lang.IllegalArgumentException: Tmp detached view should be removed from RecyclerView before it can be recycled
    • Fix Enrollment presenter tests
    • Fix SearchTEIViewModel tests
    • Fix scroll to position 0 after clicked search tei item and back to list

💥 How can it be tested?

The customizations in the app should work.

¡Important is necessary to merge related PR in SDK and create a new tag in the SDK!. This practice is new by @ifoche

💾 Requires DB migration?

  • Nope, we can just merge this branch.
  • Yes, but we need to apply it before merging this branch.
  • Yes, it's already applied.

🎨 UI changes?

  • Nope, the UI remains as beautiful as it was before!
  • Yeap, here you have some screenshots-

andresmr and others added 30 commits June 13, 2024 09:48
* chore: Update SDK to 1.10.0.1-SNAPSHOT

Signed-off-by: andresmr <[email protected]>

* chore: Update SDK to 1.10.0.1

Signed-off-by: andresmr <[email protected]>

---------

Signed-off-by: andresmr <[email protected]>
# Conflicts:
#	app/src/androidTest/java/org/dhis2/usescases/flow/teiFlow/TeiFlowTest.kt
#	app/src/androidTest/java/org/dhis2/usescases/searchte/SearchTETest.kt
#	app/src/main/res/values/strings.xml
* Update design system

Signed-off-by: andresmr <[email protected]>

* SchedulingDialogUiTest wait for idle

Signed-off-by: andresmr <[email protected]>

* SchedulingDialogUiTest wait for idle

Signed-off-by: andresmr <[email protected]>

---------

Signed-off-by: andresmr <[email protected]>
# Conflicts:
#	gradle/libs.versions.toml
* Replace Org Unit with mobile ui component

* Update designSystem version to `0.3.0-SNAPSHOT`

* Fix lint error

* Fix org unit selector robot test

* Fix `shouldMoveToNextRowWhenClickingNext` test

* Fix `selectProgramStage` test

* Fix `OrgBottomSheet` clear all btn text

* Revert CellTableTest

Signed-off-by: andresmr <[email protected]>

---------

Signed-off-by: andresmr <[email protected]>
Co-authored-by: Siddharth Agarwal <[email protected]>
…ed events (dhis2#3696)

* fix: [ANDROAPP-6101] Do not show save anyway button if there are errors, only with warnings

* ci: [ANDROAPP-6101] Ignore Compose table flaky
…nerate duplicates (dhis2#3688)

* test signed commit

* test signed commit 5

* test signed commit 6

* test signed commit 7

* test final commit

* Rename .java to .kt

* fix: [ANDROAPP-6132] Create single event enforcer, implement it throughout all new enrollment/event buttons and orgUnitDialog, migrate EventInitialActivity to kt, review sonarLint issues

* fix: [ANDROAPP-6132] remove debug comments

* fix: [ANDROAPP-6132] wrong require condition

* fix: [ANDROAPP-6132] remove deprecated date utils calls, refactor SingleEventManager to call from presenters when possible

* fix: [ANDROAPP-6132] refactor OUTreeFragment and viewmodel selected Org units check

* fix: [ANDROAPP-6132] ktlint

* fix: [ANDROAPP-6132] fix unit test

* fix: [ANDROAPP-6132] remove unused import
dhis2#3673)

* fix: [ANDROAPP-6193] App asks device location permission after granting it

Signed-off-by: Pablo <[email protected]>

* fix: check location is enabled

Signed-off-by: Pablo <[email protected]>

---------

Signed-off-by: Pablo <[email protected]>
* fix: [ANDROAPP-6169] Update gradle and kotlin version

Signed-off-by: Pablo <[email protected]>

* fix test

Signed-off-by: Pablo <[email protected]>

* wait for compose to idle in test

Signed-off-by: Pablo <[email protected]>

---------

Signed-off-by: Pablo <[email protected]>
xurxodev and others added 24 commits January 9, 2025 07:54
…_3_1_0_1

[PSI] bring last changes 3.1 0.1
…size_to_download

[PSI] configure image resize to download
…nts/bring_last_changes_3_1_0_1

# Conflicts:
#	app/build.gradle.kts
#	app/src/main/java/org/dhis2/data/service/ReservedValuesWorkerModule.kt
#	app/src/main/java/org/dhis2/usescases/enrollment/EnrollmentActivity.kt
#	app/src/main/java/org/dhis2/usescases/enrollment/EnrollmentModule.kt
#	app/src/main/java/org/dhis2/usescases/enrollment/EnrollmentPresenterImpl.kt
#	app/src/main/java/org/dhis2/usescases/enrollment/EnrollmentView.kt
#	app/src/main/java/org/dhis2/usescases/eventsWithoutRegistration/eventCapture/EventCaptureActivity.kt
#	app/src/main/java/org/dhis2/usescases/eventsWithoutRegistration/eventCapture/EventCaptureContract.kt
#	app/src/main/java/org/dhis2/usescases/eventsWithoutRegistration/eventCapture/EventCaptureModule.java
#	app/src/main/java/org/dhis2/usescases/eventsWithoutRegistration/eventCapture/EventCapturePagerAdapter.java
#	app/src/main/java/org/dhis2/usescases/eventsWithoutRegistration/eventCapture/EventCapturePresenterImpl.kt
#	app/src/main/java/org/dhis2/usescases/eventsWithoutRegistration/eventCapture/EventCaptureRepositoryImpl.java
#	app/src/main/java/org/dhis2/usescases/eventsWithoutRegistration/eventCapture/eventCaptureFragment/EventCaptureFormFragment.java
#	app/src/main/java/org/dhis2/usescases/eventsWithoutRegistration/eventCapture/eventCaptureFragment/EventCaptureFormModule.kt
#	app/src/main/java/org/dhis2/usescases/eventsWithoutRegistration/eventCapture/eventCaptureFragment/EventCaptureFormPresenter.kt
#	app/src/main/java/org/dhis2/usescases/eventsWithoutRegistration/eventInitial/EventInitialActivity.java
#	app/src/main/java/org/dhis2/usescases/login/LoginModule.kt
#	app/src/main/java/org/dhis2/usescases/main/program/ProgramFragment.kt
#	app/src/main/java/org/dhis2/usescases/main/program/ProgramModule.kt
#	app/src/main/java/org/dhis2/usescases/main/program/ProgramPresenter.kt
#	app/src/main/java/org/dhis2/usescases/programEventDetail/ProgramEventDetailRepository.kt
#	app/src/main/java/org/dhis2/usescases/programEventDetail/ProgramEventDetailRepositoryImpl.kt
#	app/src/main/java/org/dhis2/usescases/programEventDetail/eventList/EventListPresenter.kt
#	app/src/main/java/org/dhis2/usescases/searchTrackEntity/SearchRepository.java
#	app/src/main/java/org/dhis2/usescases/searchTrackEntity/SearchRepositoryImpl.java
#	app/src/main/java/org/dhis2/usescases/searchTrackEntity/SearchRepositoryImplKt.kt
#	app/src/main/java/org/dhis2/usescases/searchTrackEntity/SearchTEActivity.java
#	app/src/main/java/org/dhis2/usescases/searchTrackEntity/SearchTEIViewModel.kt
#	app/src/main/java/org/dhis2/usescases/searchTrackEntity/SearchTEModule.java
#	app/src/main/java/org/dhis2/usescases/searchTrackEntity/SearchTEPresenter.java
#	app/src/main/java/org/dhis2/usescases/searchTrackEntity/SearchTeiViewModelFactory.kt
#	app/src/main/java/org/dhis2/usescases/searchTrackEntity/listView/SearchTEList.kt
#	app/src/main/java/org/dhis2/usescases/searchTrackEntity/ui/SearchScreenConfigurator.kt
#	app/src/main/java/org/dhis2/usescases/searchTrackEntity/ui/mapper/TEICardMapper.kt
#	app/src/main/java/org/dhis2/usescases/teiDashboard/TeiDashboardMobileActivity.kt
#	app/src/main/java/org/dhis2/usescases/teiDashboard/adapters/DashboardPagerAdapter.kt
#	app/src/main/java/org/dhis2/usescases/teiDashboard/dashboardfragments/relationships/RelationshipRepositoryImpl.kt
#	app/src/main/java/org/dhis2/usescases/teiDashboard/dashboardfragments/teidata/TEIDataFragment.kt
#	app/src/main/java/org/dhis2/usescases/teiDashboard/dashboardfragments/teidata/TEIDataModule.kt
#	app/src/main/java/org/dhis2/usescases/teiDashboard/dashboardfragments/teidata/TEIDataPresenter.kt
#	app/src/main/java/org/dhis2/usescases/teiDashboard/dashboardfragments/teidata/TeiDataRepositoryImpl.kt
#	app/src/main/java/org/dhis2/usescases/teiDashboard/ui/mapper/TeiDashboardCardMapper.kt
#	app/src/main/res/layout/form_biometrics_attribute.xml
#	app/src/main/res/layout/form_biometrics_data_element.xml
#	app/src/main/res/layout/form_biometrics_register.xml
#	app/src/main/res/values-es/strings.xml
#	app/src/main/res/values/strings.xml
#	app/src/main/res/values/styles.xml
#	app/src/test/java/org/dhis2/usescases/enrollment/EnrollmentPresenterImplTest.kt
#	app/src/test/java/org/dhis2/usescases/eventsWithoutRegistration/eventCapture/EventIntegrationTest.kt
#	app/src/test/java/org/dhis2/usescases/eventsWithoutRegistration/eventCapture/eventCaptureFragment/EventCaptureFormPresenterTest.kt
#	app/src/test/java/org/dhis2/usescases/main/program/ProgramPresenterTest.kt
#	app/src/test/java/org/dhis2/usescases/searchTrackEntity/SearchTEIViewModelTest.kt
#	app/src/test/java/org/dhis2/usescases/searchTrackEntity/ui/mapper/TEICardMapperTest.kt
#	app/src/test/java/org/dhis2/usescases/teiDashboard/dashboardfragments/data/TeiDataPresenterTest.kt
#	commons/build.gradle.kts
#	commons/src/main/res/values/colors.xml
#	dhis2-android-sdk
#	form/src/main/java/org/dhis2/form/data/EnrollmentRepository.kt
#	form/src/main/java/org/dhis2/form/data/EventRepository.kt
#	form/src/main/java/org/dhis2/form/data/metadata/EnrollmentConfiguration.kt
#	form/src/main/java/org/dhis2/form/ui/DataEntryAdapter.kt
#	form/src/main/java/org/dhis2/form/ui/DataEntryViewHolderTypes.kt
#	form/src/main/java/org/dhis2/form/ui/FormView.kt
#	form/src/main/java/org/dhis2/form/ui/FormViewHolder.kt
#	form/src/main/java/org/dhis2/form/ui/LayoutProviderImpl.kt
#	form/src/main/java/org/dhis2/form/ui/provider/LayoutProvider.kt
#	form/src/main/java/org/dhis2/form/ui/provider/inputfield/DateProvider.kt
#	form/src/main/java/org/dhis2/form/ui/provider/inputfield/FieldProvider.kt
#	form/src/main/res/values/strings.xml
#	gradle/libs.versions.toml
#	settings.gradle.kts
TEI form contain a lot of biometrics customisations in enrolment presenter
To avoid duplicate logic, here hide form in landscape mode. It will be shown only in EnrollmentActivity after click on edit equal to portrait
…' into feature-simprints/bring_last_changes_3_1_0_1
java.lang.IllegalArgumentException: Tmp detached view should be removed from RecyclerView before it can be recycled
@xurxodev xurxodev marked this pull request as ready for review March 27, 2025 10:47
@ifoche ifoche changed the base branch from develop-simprints to feature-simprints/fix_cursor_bug April 2, 2025 08:02
Copy link
Member

@ifoche ifoche left a comment

Choose a reason for hiding this comment

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

thanks @xurxodev

@ifoche ifoche merged commit 9dbab57 into feature-simprints/fix_cursor_bug Apr 2, 2025
@ifoche ifoche deleted the feature-simprints/bring_last_changes_3_1_0_1 branch April 2, 2025 08:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.