Skip to content

Fix program requirement root node ordering#3294

Merged
rhysyngsun merged 2 commits into
mainfrom
nl/fix-program-requirement-root-node-ordering
Feb 17, 2026
Merged

Fix program requirement root node ordering#3294
rhysyngsun merged 2 commits into
mainfrom
nl/fix-program-requirement-root-node-ordering

Conversation

@rhysyngsun
Copy link
Copy Markdown
Collaborator

What are the relevant tickets?

Fixes https://github.com/mitodl/hq/issues/10188

Description (What does it do?)

Sets the collation of the path field to C so that paths sort correctly under all host systems.

How can this be tested?

  • Create 36 programs and try to add a 37th locally on main, it should fail
  • Checkout this branch and run the migrations
  • You should now be able to create the 37th (and more) program

@github-actions
Copy link
Copy Markdown

OpenAPI Changes

Show/hide ## Changes for v0.yaml:
## Changes for v0.yaml:


## Changes for v1.yaml:


## Changes for v2.yaml:


Unexpected changes? Ensure your branch is up-to-date with main (consider rebasing).

@jkachel jkachel self-assigned this Feb 13, 2026
@rhysyngsun rhysyngsun changed the title Nl/fix program requirement root node ordering Fix program requirement root node ordering Feb 13, 2026
Copy link
Copy Markdown
Contributor

@jkachel jkachel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@rhysyngsun rhysyngsun merged commit c2fe1c8 into main Feb 17, 2026
10 checks passed
@rhysyngsun rhysyngsun deleted the nl/fix-program-requirement-root-node-ordering branch February 17, 2026 15:00
@odlbot odlbot mentioned this pull request Feb 17, 2026
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants