Skip to content

Fix: Remove default scopes - #104

Merged
jonknapp merged 3 commits into
mainfrom
remove-default-scope
Aug 29, 2026
Merged

Fix: Remove default scopes#104
jonknapp merged 3 commits into
mainfrom
remove-default-scope

Conversation

@sean-dickinson

@sean-dickinson sean-dickinson commented Aug 29, 2026

Copy link
Copy Markdown
Collaborator

What

  • Remove the default scopes applied to models with ordering in favor of an explicit scope
  • Adds a test to assert that the content modules are ordering their links correctly

@jonknapp
jonknapp merged commit fdd3440 into main Aug 29, 2026
5 checks passed
@jonknapp
jonknapp deleted the remove-default-scope branch August 29, 2026 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jonknapp jonknapp jonknapp approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@sean-dickinson @jonknapp