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.

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

Optimize location hierarchy #87

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Optimize location hierarchy #87

wants to merge 6 commits into from

Commits on Sep 24, 2024

  1. Add extra checks on AdminLevel filters

    @lincmba
    lincmba committed Sep 24, 2024
    Configuration menu
    Copy the full SHA
    de7868b View commit details
    Browse the repository at this point in the history

  2. Optimize non identifier request 

     Instead of fetching entire practitionerdetails, just get the locations.
    @lincmba
    lincmba committed Sep 24, 2024
    Configuration menu
    Copy the full SHA
    ec06253 View commit details
    Browse the repository at this point in the history

Commits on Sep 25, 2024

  1. Optimize further: 

     - replace for loops with streams
 - use sets instead of lists to avoid duplication
 - switch from using DFS to BFS in getting locations deep in the hierarchy
    @lincmba
    lincmba committed Sep 25, 2024
    Configuration menu
    Copy the full SHA
    77363cf View commit details
    Browse the repository at this point in the history

  2. Optimize further: 

     - switch from for loop to streams in filterLocationsByAdminLevels
    @lincmba
    lincmba committed Sep 25, 2024
    Configuration menu
    Copy the full SHA
    d5fd24f View commit details
    Browse the repository at this point in the history

  3. Add tests

    @lincmba
    lincmba committed Sep 25, 2024
    Configuration menu
    Copy the full SHA
    fd7909a View commit details
    Browse the repository at this point in the history

Commits on Oct 1, 2024

  1. Functions reverted to work as before

    @lincmba
    lincmba committed Oct 1, 2024
    Configuration menu
    Copy the full SHA
    f8a344a View commit details
    Browse the repository at this point in the history