Skip to content

WMR dataValues migration - updates#1009

Merged
MiquelAdell merged 31 commits intodevelopmentfrom
feature/wmr-sync-updates
Oct 22, 2025
Merged

WMR dataValues migration - updates#1009
MiquelAdell merged 31 commits intodevelopmentfrom
feature/wmr-sync-updates

Conversation

@MatiasArriola
Copy link
Copy Markdown
Contributor

@MatiasArriola MatiasArriola commented May 27, 2025

📌 References

📝 Implementation

This is a continuation of the work from #969

  • Merge from development
  • First page changes
    • Check if dataSet exists.
    • Import dataset for the root OrgUnit
    • Check if dataStore config exists.
    • Import dataStore
    • Error handling. Still allow to download and manually import in case of errors.
    • Remove tutorial button
  • Remove OrgUnits warning, and filter OrgUnit tree instead Remove Org unit selection step, use root org unit
  • For mapping local dataSets, filter the source list to show only dataSets from the root OrgUnit
  • Implement remote instance target form
  • Map OrgUnits in local to target Provide OrgUnit ID in final step
  • Implement Sync with remote instance

Extras:

  • Review if consistency checks are needed for imports. Now it just checks that the dataSet exists, or the dataStore key exists - No consistency checks for now
  • Review correct value for the dataStore key - Used WMR-MAL
  • Change usage of useWmrSettings (it shouldn't retrieve the dataset before completion of step 1)
  • Enable/Disable wizard buttons depending on state
  • Review if needed to implement a better remote OrgUnit picker instead of the id TextField
  • Review if needed to implement more detailed report after sync (detail imported dataElements, ommited, error msgs, etc)

Changes to core components:

📹 Screenshots/Screen capture

🔥 Is there anything the reviewer should know to test it?

📑 Others

@ifoche
Copy link
Copy Markdown
Member

ifoche commented May 27, 2025

refactor: GeneralInfoForm (instance form) extract onSaved and showMetadataMapping props to make it reusable
…ore advancing steps. Move sync and check to a single step.

fix: dataentry loading, use waitFor instead of fixed timeouts
refactor: use wmr context for wizard. Add useSyncLocalWmr hook, remove useWmrSettings, move load of settings into context and call in 2nd step.
refactor: extract RemoteInstanceSelector
@MatiasArriola MatiasArriola changed the base branch from feature/wmr-sync to development June 11, 2025 12:04
@MatiasArriola MatiasArriola changed the base branch from development to feature/wmr-sync June 11, 2025 12:04
@MatiasArriola MatiasArriola changed the base branch from feature/wmr-sync to development October 21, 2025 14:50
@MatiasArriola MatiasArriola marked this pull request as ready for review October 21, 2025 14:50
@bundlemon
Copy link
Copy Markdown

bundlemon Bot commented Oct 22, 2025

BundleMon

No change in files bundle size

Groups updated (1)
Status Path Size Limits
Build Folder
./**/*
2.78MB (+1.02MB +57.73%) +20%

Final result: ❌

View report in BundleMon website ➡️


Current branch size history | Target branch size history

@MiquelAdell MiquelAdell merged commit 66bd238 into development Oct 22, 2025
3 of 6 checks passed
@MiquelAdell MiquelAdell deleted the feature/wmr-sync-updates branch October 22, 2025 11:54
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.

4 participants