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

Perf: Better response times from api/public/v1/courses #4616

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

Perf: Better response times from api/public/v1/courses #4616

wants to merge 8 commits into from
+69 −7

Commits on Oct 31, 2024

  1. Reduce number of queries in the courses api

    @inulty-dfe
    inulty-dfe committed Oct 31, 2024
    Configuration menu
    Copy the full SHA
    ac7a654 View commit details
    Browse the repository at this point in the history

  2. Cache the course count in the api

    @inulty-dfe
    inulty-dfe committed Oct 31, 2024
    Configuration menu
    Copy the full SHA
    fe64c32 View commit details
    Browse the repository at this point in the history

  3. Cache year 

      Need to cache against all params
    @inulty-dfe
    inulty-dfe committed Oct 31, 2024
    Configuration menu
    Copy the full SHA
    ca45871 View commit details
    Browse the repository at this point in the history

  4. Filter out enrichments where the last_published_timestamp_utc is nil

    @inulty-dfe
    inulty-dfe committed Oct 31, 2024
    Configuration menu
    Copy the full SHA
    0e5d49f View commit details
    Browse the repository at this point in the history

  5. Add test for when course.enrichments are not loaded

    @inulty-dfe
    inulty-dfe committed Oct 31, 2024
    Configuration menu
    Copy the full SHA
    8562649 View commit details
    Browse the repository at this point in the history

Commits on Nov 1, 2024

  1. Add spec for Course#last_published_at

    @inulty-dfe
    inulty-dfe committed Nov 1, 2024
    Configuration menu
    Copy the full SHA
    dc7f15a View commit details
    Browse the repository at this point in the history

  2. Specify that api/courses params are permitted now

    @inulty-dfe
    inulty-dfe committed Nov 1, 2024
    Configuration menu
    Copy the full SHA
    acc18eb View commit details
    Browse the repository at this point in the history

  3. Remove no_cache api param

    @inulty-dfe
    inulty-dfe committed Nov 1, 2024
    Configuration menu
    Copy the full SHA
    57a9250 View commit details
    Browse the repository at this point in the history