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

fix: cannot get TE if there are no tracker programs DHIS2-18991 #19930

Merged
merged 1 commit into from
Feb 14, 2025

Conversation

teleivo
Copy link
Contributor

@teleivo teleivo commented Feb 14, 2025

cherry-pick of #19918

https://dhis2.atlassian.net/browse/DHIS2-18991 is the same bug as https://dhis2.atlassian.net/browse/DHIS2-17959 except it manifests only in tracker exporter as tracker is not part of metadata sync in 42.

So a user requesting TEs

  • with no access to any tracker program
  • or on a system without tracker programs

would get a 500 due to invalid SQL.

@teleivo teleivo marked this pull request as ready for review February 14, 2025 08:00
@teleivo teleivo requested review from a team as code owners February 14, 2025 08:00
Copy link
Contributor

@radnov radnov left a comment

Choose a reason for hiding this comment

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

Docker Compose changes look good to me. 👍

@teleivo teleivo enabled auto-merge (squash) February 14, 2025 09:43
@teleivo teleivo disabled auto-merge February 14, 2025 09:44
@teleivo teleivo enabled auto-merge (squash) February 14, 2025 10:49
@teleivo teleivo merged commit 72925ce into master Feb 14, 2025
16 checks passed
@teleivo teleivo deleted the DHIS2-18991 branch February 14, 2025 11:01
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