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

Reduce number of queries required for content tagging endpoints #629

Closed
wants to merge 7 commits into from
Closed

Reduce number of queries required for content tagging endpoints #629

wants to merge 7 commits into from

Commits on Feb 7, 2024

  1. test: fixes "list taxonomies for org" test URL 

    Was requesting org=$orgA instead of org=orgA, and so wasn't testing the
queries we needed to test.

(cherry picked from commit 0a3ada3)
    @pomegranited
    pomegranited committed Feb 7, 2024
    Configuration menu
    Copy the full SHA
    f95d05c View commit details
    Browse the repository at this point in the history

  2. fix: filter list of taxonomies by org without fetching the Organization 

    (cherry picked from commit 8e7823e)

fix: lint

(cherry picked from commit 8a0d598)
    @pomegranited
    pomegranited committed Feb 7, 2024
    Configuration menu
    Copy the full SHA
    3f20211 View commit details
    Browse the repository at this point in the history

  3. fix: prefetch tag_set to reduce queries 

    (cherry picked from commit 3057ccf)
    @pomegranited
    pomegranited committed Feb 7, 2024
    Configuration menu
    Copy the full SHA
    eccf960 View commit details
    Browse the repository at this point in the history

  4. fix: prefetch taxonomyorg orgs 

    (cherry picked from commit 6e094ef)
    @pomegranited
    pomegranited committed Feb 7, 2024
    Configuration menu
    Copy the full SHA
    9fd3c75 View commit details
    Browse the repository at this point in the history

  5. fix: use openedx-learning branch from pr#157 

    which reduces the query count.

TO DO: once PR is merged and tagged, update the requirement here.
    @pomegranited
    pomegranited committed Feb 7, 2024
    Configuration menu
    Copy the full SHA
    af6e41d View commit details
    Browse the repository at this point in the history

  6. fix: iterate over prefetched taxonomyorgs instead of filtering 

    to avoid re-querying.

Changes TaxonomyOrg.get_organizations to return the "all orgs" flag too,
which makes it useful in the rules.
    @pomegranited
    pomegranited committed Feb 7, 2024
    Configuration menu
    Copy the full SHA
    286230a View commit details
    Browse the repository at this point in the history

  7. fix: Adds a check for the TaxonomyOrg.rel_type when serializing orgs

    @pomegranited
    pomegranited committed Feb 7, 2024
    Configuration menu
    Copy the full SHA
    b372861 View commit details
    Browse the repository at this point in the history