Skip to content

fix: [DHIS2-15903] prod env fix for Capture app caching logic#633

Merged
JoakimSM merged 2 commits intomasterfrom
DHIS2-15903-prod-fix
Sep 16, 2025
Merged

fix: [DHIS2-15903] prod env fix for Capture app caching logic#633
JoakimSM merged 2 commits intomasterfrom
DHIS2-15903-prod-fix

Conversation

@JoakimSM
Copy link
Copy Markdown
Member

We need the ABSOLUTE base url for the instance when getting the Capture app databases. We relied on the baseUrl property of the root object returned by useConfig (from app-runtime) for this, but it is not reliable as it will return a relative path for: some backend versions (v41 and below) / app-shell versions.

We will use the contextPath set by the backend instead.

@dhis2-bot
Copy link
Copy Markdown
Contributor

dhis2-bot commented Aug 28, 2025

🚀 Deployed on https://pr-633--dhis2-cache-cleaner.netlify.app

@dhis2-bot dhis2-bot temporarily deployed to netlify August 28, 2025 14:05 Inactive
@JoakimSM JoakimSM force-pushed the DHIS2-15903-prod-fix branch from 2c1ad74 to 7639b31 Compare August 28, 2025 14:06
@dhis2-bot dhis2-bot temporarily deployed to netlify August 28, 2025 14:07 Inactive
@sonarqubecloud
Copy link
Copy Markdown

@dhis2-bot dhis2-bot temporarily deployed to netlify August 29, 2025 12:29 Inactive
@JoakimSM JoakimSM marked this pull request as ready for review August 29, 2025 12:33
@JoakimSM JoakimSM requested review from a team as code owners August 29, 2025 12:33
@JoakimSM JoakimSM merged commit 9f37981 into master Sep 16, 2025
10 checks passed
@JoakimSM JoakimSM deleted the DHIS2-15903-prod-fix branch September 16, 2025 07:46
dhis2-bot added a commit that referenced this pull request Sep 16, 2025
## [100.2.1](v100.2.0...v100.2.1) (2025-09-16)

### Bug Fixes

* [DHIS2-15903] prod env fix for Capture app caching logic ([#633](#633)) ([9f37981](9f37981))
@dhis2-bot
Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 100.2.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants