Skip to content

Conversation

shubham7109
Copy link
Collaborator

@shubham7109 shubham7109 commented Jul 10, 2025

Related to issue: #6138

Description:

PR to merge feature branch Offline map areas to v.next.

Pre-merge Checklist

shubham7109 and others added 30 commits May 1, 2025 09:52
* Changes to initialize preplanned map area

* update apiDump file

* add newline and copyright header

* remove initializer for modifier

* address code review comments

* remove spacer
* Changes to enhance the preplanned map areas list

* Add preplanned areas state, offlineMapInfo

* add newline
add strings.xml to t9nmanifest.txt

* update api file
remove composableSingletons

* address code review comments
* Add backing properties

* remove commented code

* remove unused imports
* WIP

* pair programming progress

* clean up app code

* Make parent directory if absent

* more pair programming progress

* Monday refactor work, updated workmanager to observe UUID, updated OfflineMapState to initialize with Context, changed microapp to use ViewModel

* add newline and TODO

* add portalItemId of the webmap's portalItem to the file download path

---------

Co-authored-by: Shubham Sharma <shubhamsharma@esri.com>
…odel scope (#856)

* Add fix so that we always run the workManagerRepository fun in the viewModel scope
so the long running collect operations survive orientation changes

* update api file
remove unused code

* address code review comments

* update api file

* removed unused import
* Add fix so that we always run the workManagerRepository fun in the viewModel scope
so the long running collect operations survive orientation changes

* update api file
remove unused code

* WIP

* WIP

* some clean up and added currently open message

* remove unused fun
update api file

* update api file

* remove unused import

* address code review comments

* revert notification changes

* move WorkManagerRepository to workmanager package
* Add fix so that we always run the workManagerRepository fun in the viewModel scope
so the long running collect operations survive orientation changes

* update api file
remove unused code

* WIP

* WIP

* some clean up and added currently open message

* remove unused fun
update api file

* update api file

* remove unused import

* Add changes to toggle between online and offline map

* remove unused code

* refactor onSelectionChangedListerner name in PreplannedMapAreaState
* WIP

* WIP

* Changes to show map area details in a modal bottomsheet

* copy info file only if it exists in the pending directory

* add scroll to bottomsheet
add logic to delete offlinemapinfo along with the last map area to be deleted

* refactor MapAreaDetailsScreen

* add newline

* update build.gradle to omit composableSingleton for MapAreaDetailsScreen

* cancel scope after initialization

* Add remove download methods to OfflineRepository
rename delete download button Text

* update offlineMapInfos

* address code review comments

* address code review comments

* apply feedback
#883)

* Changes to make OfflineRepository Singleton, to support single instance of OfflineMapInfos to work as a SnapShotStateList

* some clean up

* update api file
)

* WIP

* clean up

* remove Experimental api usage
add style for ComponentDialog

* update build.gradle

* update offline.api
Add Modifications to copyright
* refactor common code

* WIP

* add newline

* update build.gradle

* address code review comments
* update check to determine if the device is offline

* update logic

* address code review feedback

* update micro app
upate offlineMapState
@shubham7109 shubham7109 self-assigned this Jul 10, 2025
@shubham7109 shubham7109 requested a review from puneet-pdx July 10, 2025 16:12
Copy link
Collaborator

@puneet-pdx puneet-pdx left a comment

Choose a reason for hiding this comment

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

@shubham7109 LGTM 👍🏽

@shubham7109 shubham7109 merged commit bf8a094 into v.next Jul 11, 2025
@shubham7109 shubham7109 deleted the feature-branches/offline-map-areas branch July 11, 2025 16:56
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.

2 participants