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

Prepopulate unscheduled sessions #1942

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

Prepopulate unscheduled sessions #1942

wants to merge 5 commits into from

Commits on Oct 8, 2024

  1. Add UnscheduledSessionsFactory 

    This adds a class which populates the unscheduled sessions for each
team, either creating missing sessions or destroying any sessions that
shouldn't exist.
    @thomasleese
    thomasleese committed Oct 8, 2024
    Configuration menu
    Copy the full SHA
    1256526 View commit details
    Browse the repository at this point in the history

  2. Create unscheduled sessions as part of seeds 

    When seeding the database for the first time we should create any
unscheduled sessions so they're displayed to the user.
    @thomasleese
    thomasleese committed Oct 8, 2024
    Configuration menu
    Copy the full SHA
    93923c0 View commit details
    Browse the repository at this point in the history

  3. Remove ability to create unscheduled sessions dynamically 

    Instead these sessions will created when schools are added to teams and
when the year comes around.
    @thomasleese
    thomasleese committed Oct 8, 2024
    Configuration menu
    Copy the full SHA
    bada7c4 View commit details
    Browse the repository at this point in the history

  4. Create unscheduled sessions after creating a location 

    When adding a location to a team we should also create any unscheduled
sessions for the year.
    @thomasleese
    thomasleese committed Oct 8, 2024
    Configuration menu
    Copy the full SHA
    3d04870 View commit details
    Browse the repository at this point in the history

  5. Don't automatically set programmes 

    When creating sessions, we don't need to set the programmes anymore as
they're set as part of the factory or when importing records for the
first time.

I've also added a validation to ensure a session has programmes.
    @thomasleese
    thomasleese committed Oct 8, 2024
    Configuration menu
    Copy the full SHA
    ea540b8 View commit details
    Browse the repository at this point in the history