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: [DHIS2-18958] Add loading functionality to Search in all programs button #3965

Conversation

henrikmv
Copy link
Contributor

@henrikmv henrikmv commented Feb 11, 2025

DHIS2-18958:

When the button is clicked, it enters a loading state. It will remain in this state until the Redux variable otherResults is defined. This ensures that the button stays in a loading state while the search is in progress and updates only when results are available.

A loading spinner is also implemented when navigating between pages while searching across all programs.

@henrikmv henrikmv marked this pull request as ready for review February 11, 2025 12:30
@henrikmv henrikmv requested a review from a team as a code owner February 11, 2025 12:30
Copy link
Contributor

@simonadomnisoru simonadomnisoru left a comment

Choose a reason for hiding this comment

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

Hi @henrikmv,
The Search all programs button loading functionality looks great 👏 . I just was wondering if we should add a loading spinner when changing pages too (just like we do when searching in a specific program)? This is not in the Jira ticket but it feels related and may also be an easy fix. Any thoughts?
Thanks!

@henrikmv
Copy link
Contributor Author

Hi @henrikmv, The Search all programs button loading functionality looks great 👏 . I just was wondering if we should add a loading spinner when changing pages too (just like we do when searching in a specific program)? This is not in the Jira ticket but it feels related and may also be an easy fix. Any thoughts? Thanks!

Thanks Simona! That is a good suggestion and has now been implemented.

Copy link

github-actions bot commented Feb 12, 2025

Copy link

@geethaalwan geethaalwan left a comment

Choose a reason for hiding this comment

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

Tested successfully on 2.42,2.41.4,2.40.8,2.39.9 versions

@henrikmv henrikmv added testing and removed preview labels Feb 13, 2025
@henrikmv henrikmv merged commit 6486942 into master Feb 17, 2025
52 checks passed
@henrikmv henrikmv deleted the hv/fix/DHIS2-18958_MissingLoadingSpinnerForSearchAllProgramsButton branch February 17, 2025 15:46
dhis2-bot added a commit that referenced this pull request Feb 17, 2025
## [101.27.1](v101.27.0...v101.27.1) (2025-02-17)

### Bug Fixes

* [DHIS2-18958] Add loading functionality to Search in all programs button ([#3965](#3965)) ([6486942](6486942))
@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 101.27.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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants