Skip to content

Migrate to Order Levels#589

Merged
cihandeniz merged 26 commits into
mouseless:epic/convention-order-layerizationfrom
cihandeniz:migrate-to-order-levels
Jun 8, 2026
Merged

Migrate to Order Levels#589
cihandeniz merged 26 commits into
mouseless:epic/convention-order-layerizationfrom
cihandeniz:migrate-to-order-levels

Conversation

@cihandeniz

@cihandeniz cihandeniz commented May 30, 2026 •
edited by dncsvr
Loading

Copy link
Copy Markdown
Member

Refactor features to use new Level system.

Tasks

  • Remove older order system
  • Migrate theme to new order system
  • Migrate relative min/max to new order system
  • Fix build errors
  • Fix generate errors
  • Fix page output changes
    • generated files are not committed, compare with old files
  • Fix hardcoded Order default values to be configurable
    • ex. DomainModelConventionCollection defaults to "Add" or "Configure",
      this should not be hardcoded
  • Fix changes in api routes
    • route-samples.kdl
  • Update ManagingOrders and ApplyingConventions tests for
    new order design
    • Base, Level or Extension null should have proper exception messages
  • Revise TODO notes and use suggested orders where possible

Additional Task

  • ModelCollection index should set value to readonly list

@cihandeniz cihandeniz self-assigned this May 30, 2026
@cihandeniz cihandeniz mentioned this pull request May 30, 2026
Merged
12 tasks
cihandeniz
cihandeniz commented Jun 2, 2026
View reviewed changes
Comment thread core/src/Baked/Domain/Configuration/Order.cs Outdated
Comment thread core/src/Baked/CodingStyle/UseBuiltInTypes/UseBuiltInTypesCodingStyleFeature.cs Outdated
Comment thread core/src/Baked/CodingStyle/UseNullableTypes/UseNullableTypesCodingStyleFeature.cs Outdated
Comment thread core/src/Baked/Domain/Configuration/DomainModelBuilderOptions.cs Outdated
Comment thread core/src/Baked/Domain/DomainModelConventionCollection.cs Outdated
Comment thread core/src/Baked/Ux/DataTableDefaults/DataTableDefaultsUxFeature.cs Outdated
Comment thread core/src/Baked/Ux/PropertiesAsFieldset/PropertiesAsFieldsetUxFeature.cs Outdated
Comment thread core/test/Baked.Test.Specs/Domain/ApplyingConventions.cs Outdated
Comment thread core/test/Baked.Test.Specs/Domain/ManagingOrders.cs Outdated
Comment thread core/playground/Baked.Playground.Application/.baked/rest-api/route-samples.kdl Outdated
dncsvr added 13 commits June 3, 2026 14:00
@dncsvr
edit add locate action order to fix api route changes
1cdc9f7
@dncsvr
change index value to immutable list
b2644d0
@dncsvr
fix managing orders tests
4c14680
@dncsvr
fix failing tests
c094377 
- uncomment diagnostics in DomainModelConventionCollection
- minor format edits
@dncsvr
use Min for builtin types and nullable types feature convention orders
ff52f18 
- remove todo notes
@dncsvr
add applying conventions test case for full base level extension orde…
744002c 
…ring

- minor edits
@dncsvr
use Max for ActionModel and ParameterModel attribute conventions
f0c9eed
@dncsvr
use Max order in LocatableExtensions feature
8d3431b
@dncsvr
add default values for convention matrix and default leve
dbc2843 
- revert changes in DomianModelExtensions
@dncsvr
add OrderMatrix to manage convention defaults
2fd7e9b 
- rename ConventionMatrix to ConventionOrderMatrix
@dncsvr
refactor OrderMatrix
13663b7
@dncsvr
minor edits
715fce2
@dncsvr
move domain model convention collection extensions to Business Extens…
87a5e2d 
…ions
@dncsvr dncsvr marked this pull request as ready for review June 8, 2026 12:01
@cihandeniz cihandeniz requested a review from dncsvr June 8, 2026 12:30
@cihandeniz cihandeniz merged commit a17f850 into mouseless:epic/convention-order-layerization Jun 8, 2026
4 checks passed
@cihandeniz cihandeniz deleted the migrate-to-order-levels branch June 8, 2026 12:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dncsvr dncsvr dncsvr approved these changes

Assignees

@cihandeniz cihandeniz

@dncsvr dncsvr

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@cihandeniz @dncsvr