Skip to content

Update SchoolPresenceExemption to allow fee paying courses - #6316

Open
inulty-dfe wants to merge 6 commits into
mainfrom
im/no-school-courses-can-be-fee-paying
Open

Update SchoolPresenceExemption to allow fee paying courses#6316
inulty-dfe wants to merge 6 commits into
mainfrom
im/no-school-courses-can-be-fee-paying

Conversation

@inulty-dfe

@inulty-dfe inulty-dfe commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Context

We permit salary and apprenticeship courses to publish without course schools attached, if the course has been exempted by Support.

We are now going to change it so that fee paying courses are allowed to behave the same way. This means that all funding types can be exempt from attaching course schools in future.

Changes proposed in this pull request

  • Allow Support users to exempt any course from requiring schools attached
  • Relax the validation for exempt courses to permit any funding type
  • Fix a flakey test regarding GCSE Grade

Guidance to review

Checklist

  • I have moved hard-coded strings to locale files.
  • I have removed the usage of data-qa attributes in HTML files and updated the corresponding tests.

@inulty-dfe

Copy link
Copy Markdown
Contributor Author

@inulty-dfe
inulty-dfe force-pushed the im/no-school-courses-can-be-fee-paying branch from 42668a9 to 599fad0 Compare August 12, 2026 15:46
@inulty-dfe
inulty-dfe marked this pull request as ready for review August 13, 2026 11:18
@inulty-dfe
inulty-dfe requested a review from a team as a code owner August 13, 2026 11:18
  Just one provider code exists that renders a different GCSE grade
  requirement to the rest of the providers.

  We should avoid generating provider with code I30 by chance and
  require it to be generated with a dedicated factory.
@inulty-dfe
inulty-dfe force-pushed the im/no-school-courses-can-be-fee-paying branch from 7c49e6a to 8c0d3ab Compare August 13, 2026 12:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant