Skip to content

Migrate Instances to routes #1032

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 41 commits into
base: development
Choose a base branch
from
Draft

Migrate Instances to routes #1032

wants to merge 41 commits into from

Conversation

@xurxodev
Copy link
Contributor

@xurxodev xurxodev commented Oct 16, 2025 •
edited
Loading

📌 References

📝 Implementation

  • List instances reading from routes
  • Create/edit instance in routes
  • Delete instances from routes
  • Create migration from data store to routes
  • Test instance connection using routes
  • Update metadata repository to request using routes when its necessary
  • Update aggregated repository to request using routes when its necessary
  • Update events repository to request using routes when its necessary
  • Remove create local instance in dhis2 if not exists
  • Migrate request get version
  • Fix manual calls without repository to target instance using d2api to use routes
  • Implement auth in instances using token

Bugs reported to dhis2
https://dhis2.atlassian.net/browse/DHIS2-20010
https://dhis2.atlassian.net/browse/DHIS2-20130
https://dhis2.atlassian.net/browse/DHIS2-20311

  • Complete testing when dhis2 has solved the reported bugs

📹 Screenshots/Screen capture

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

📑 Others

  • Any change in the GUI library? If so, what branch/PR?

  • Any change in the D2 Api? If so, what branch/PR?

xurxodev added 30 commits June 25, 2025 11:34
@xurxodev
Retrieve instances of routes endpoint
346c7ac
@xurxodev
Remove create local instance if not exists
71176a1
@xurxodev
test metadata repository using routes
e773d9d
@xurxodev
Get instance id to format the url
8fea8b6
@xurxodev
Fix bug to create instance as route in dhis2
dae48cd
@xurxodev
Fix update instance as route
f70b8c9
@xurxodev
Implement delete instance using routes
e73a55d
@xurxodev
Avoid request to routes endpoint using the same repository
c9f9843 
- to basic use case reuse the same instance repository with cache
@xurxodev
Remove encryptionKey.
ba884a5 
- now is not necessary using routes
@xurxodev
Remove non used lib to encrypt password
2b5f07c
@xurxodev
Fix tests
f8840ab
@xurxodev
Creation migration for instances from data store to routes
8ab7daa
@xurxodev
Fix bug to run migration
b01d829
@xurxodev
Fix test connection using routes
70f493a
@xurxodev
Avoid tu use getVersion of the repository
f8de771
@xurxodev
Mark getVersion in the repository as deprecated
8b9e876
@xurxodev
Remove getVersion from instance repository
2b6d3c9
@xurxodev
Avoid to use api requesting directly to destination without routes
f6499ff 
- Remove getApi from InstanceRepository
- Mark GetInstanceApiUseCase as deprecated for the future
@xurxodev
Avoid to request responsibles to different instance than local
16feade
@xurxodev
Move getOrgUnitRoots from instance repository to metadata repository
a44e647
@xurxodev
Hide test connection button in creation mode
0d3709e
@xurxodev
move getOrgUnitRoots from instances to metadata in composition root
6831bbe
@xurxodev
Remove getBaseUrl from InstanceRepository
89eb16e
@xurxodev
getBaseUrl from InstanceRepository
dab46d0
@xurxodev
Remoce unused repository
1da89a8
@xurxodev
Add TODO comments
69af870
@xurxodev
Use routes to invoke dataStore if required
42c3aa9
@xurxodev
Use routes in events, aggregated and Teis if required
b3b856e
@xurxodev
Simplify instantiation of userRepository using always localInstance
6f22246
@xurxodev
Fix aggregated and events integration tests
9e979fc
@xurxodev xurxodev changed the title Feature/routes as instances Migrate Instances to routes Oct 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@xurxodev