Skip to content

Issue / Locatability / ID Configuration As Feature#421

Merged
dncsvr merged 63 commits intomouseless:epic/locatabilityfrom
dncsvr:issue/locatability/id-configuration-as-feature
Jan 20, 2026
Merged

Issue / Locatability / ID Configuration As Feature#421
dncsvr merged 63 commits intomouseless:epic/locatabilityfrom
dncsvr:issue/locatability/id-configuration-as-feature

Conversation

@dncsvr
Copy link
Copy Markdown
Member

@dncsvr dncsvr commented Jan 13, 2026 •
edited
Loading

Move identitiy configuration to a seperate feature to make
it overridable

Tasks

  • Add Id struct to Business.Abstractions
    • Implement IParsable<Id> for model binding
  • Add a Id Coding Style feature for identity configuration
    • Add implementation and extensions
    • Add json converter to serialize "id": string
    • Configure swagger schema to display "id": string
  • Move IdAttribute to Business
    • Add RouteName data
    • Add OrmConfig data
  • Use property with IdAttribute in other features
    • FindTargetUsingQueryContextConvention
    • SingleByIdConvention
    • TargetEntityExtensionFromRouteConvention
    • LookupEntityByIdConvention
    • LookupEntitiesByIdsConvention
    • ConvertToId and ConvertToIds helper
    • BuildSingleBy helper
    • RecordNotFoundException
    • DefaultThemeFeature route attribute configuration
  • Remove SingleByUnique coding style feature and related samples, tests
    and features
    • SingleByUnique feature and tests
    • Entity, SingleByEnum and SingleByUnique methods and tests
    • TargetEntityExtensionsFromRouteByUniquePropertiesConvention
  • Update samples
  • Update documentation

Additional Tasks

  • Fix MySql database feature not working

@dncsvr dncsvr self-assigned this Jan 13, 2026
@cihandeniz cihandeniz changed the title Issue / Locatability / Id Configuration As Feature Issue / Locatability / ID Configuration As Feature Jan 13, 2026
dncsvr added 23 commits January 13, 2026 11:33
@dncsvr
add GuidIdAsIdentity coding style feature
ef2ee16 
- remove identity configuration from `AutoMap` feature
@dncsvr
move id tests to a coding style test fixture
8dae495
@dncsvr
minor edits
9d6377f
@dncsvr
move IdAttribute to Business abstractions
0dcc5b2
@dncsvr
start refactoring features to use property with id attribute
c6c3146
@dncsvr
add Id feature and GuidId implementation
82cf38b 
- remove `GuidIdAsIdentity` coding style feature
- update documentation
@dncsvr
fix format
40285cb
@dncsvr
refactor features to utilize IdAttribute
85c62f5 
- add `Type` and `Key` properties to IdAttribute
- add type and idSuffix parameters to `ConvertToId` helpers
@dncsvr
change BuildSingleBy property parameter to be required
77df729 
- minor edits
@dncsvr
start implementing custom Id struct !!! TESTS AND RUN FAILING !!!
24475d6 
- use `Id` in entities
@dncsvr
fix auto persistence model id configuration !!! TESTS FAILING !!!
457415e 
- add json converter for Id
- add schema filter for Id
- move id tests to Id feature folder
@dncsvr
begin removing single by unique coding style, related samples and tests
7ccf4de
@dncsvr
fix failing tests - use map type instead of schema filter for Id
eb47038 
- ignore `RoutingEntitySubclasses` tests
@dncsvr
add comments for swagger configuration
d03c920
@dncsvr
minor edits
1c14439
@dncsvr
move common configurations back to AutoMapOrmFeature
8b8d25a 
- add `IdAttribute` convention to `RichTransient` feature
@dncsvr
minor format edits
d125667
@dncsvr
minor refactoring
6a7bee7 
- change Id to record struct and make constructor private
- improve GuidIdUserType
- add test for `Id`
@dncsvr
fix mysql connection string
e63b0f2
@dncsvr
update unreleased and documentation
22c4ea8
@dncsvr
use Orm.Id type in samples project for ids
01429f5
@dncsvr
use nhibernate generator in GuidIdGenerator
5e936be 
- remove TODO comments from GuidIdUserType
@dncsvr
remove empty line
9f22730
@dncsvr dncsvr requested a review from cihandeniz January 16, 2026 10:17
@dncsvr dncsvr marked this pull request as ready for review January 16, 2026 10:17
dncsvr added 2 commits January 16, 2026 19:04
@dncsvr
review changes
6327236 
- add `TryGetıdentifier` extension for TypeModel and return `IdentifierInfo`
- rename `Id` attribute to `Identifier`
- refactor features and conventions
@dncsvr
fix format
2f079d1
dncsvr added 5 commits January 19, 2026 18:59
@dncsvr
review changes
e416cb8 
- rename IdentifierInfo to IdInfo
- move GetIdInfo helpers to Business feature
@dncsvr
review changes
6c6cb84 
- change GuidIdGenerator to extend GuidGenerator
- remove TypeCheck from GuidIdUserType
@dncsvr
rename identifier variable names to idInfo
c44aaa9
@dncsvr
minor renme
54043c4
@dncsvr
review changes
e9f80e4 
- change `ConvertToId` to use `IdInfo` as parameter
cihandeniz
cihandeniz reviewed Jan 19, 2026
View reviewed changes
Comment thread core/src/Baked/CodingStyle/Id/GuidIdGenerator.cs Outdated
Comment thread core/src/Baked/CodingStyle/Id/GuidIdGenerator.cs Outdated
Comment thread core/src/Baked/CodingStyle/Id/GuidIdUserType.cs Outdated
Comment thread core/src/Baked/CodingStyle/Id/GuidIdUserType.cs Outdated
Comment thread core/src/Baked/CodingStyle/Id/GuidIdUserType.cs Outdated
Comment thread unreleased.md Outdated
Comment thread core/src/Baked/Orm/OrmExtensions.cs Outdated
Comment thread core/src/Baked/CodingStyle/RichTransient/RichTransientCodingStyleFeature.cs Outdated
Comment thread core/src/Baked/CodingStyle/RichEntity/FindTargetUsingQueryContextConvention.cs Outdated
Comment thread core/src/Baked/CodingStyle/RichEntity/FindTargetUsingQueryContextConvention.cs Outdated
cihandeniz
cihandeniz reviewed Jan 19, 2026
View reviewed changes
Comment thread core/src/Baked/Orm/AutoMap/LookupEntitiesByIdsConvention.cs Outdated
dncsvr added 4 commits January 19, 2026 19:31
@dncsvr
review changes
9c357ec 
- rename `GuidId` to `IdGuid`
- use Parse instead of TryParse in user type
- remove empty lines
- remove unnecessary Kebaberize
@dncsvr
review changes
6a86a95 
- improve Id coding style documentation
- update auto map orm feature documentation
- update unreleased
@dncsvr
review changes
16376b6 
- restrict to Id typed properties as well as Id named properties
- update rich transient samples to use `Business.Id`
@dncsvr
review changes
2ee8e4f 
- move id tests to coding style folder
- validate identifier generator type
@dncsvr dncsvr requested a review from cihandeniz January 20, 2026 12:35
dncsvr added 9 commits January 20, 2026 16:00
@dncsvr
minor edits
0fedef8
@dncsvr
fix typo
ca30dd5
@dncsvr
review changes
c0dd248 
- rename `IdInfo.Name` to `IdInfo.PropertyName`
- use explicit Camelize for target name
@dncsvr
review changes
e5bb23f 
- change id type of sample records to `Id`
- minor format edits
@dncsvr
review changes
04d444f 
- minor format edits
@dncsvr
review changes
b82ff28 
- move TryGetIdInfo above
@dncsvr
review changes
6052bf0 
- refactor Guid id tests assertion
- minor format edits
@dncsvr
review changes
1e53379 
- update Id coding style documentation
@dncsvr
review changes
3dd9793 
- update id coding style entity sample documentation
cihandeniz
cihandeniz reviewed Jan 20, 2026
View reviewed changes
Comment thread core/playground/Baked.Playground.Application/Theme/Custom/CustomThemeFeature.cs Outdated
Comment thread core/src/Baked/CodingStyle/Id/IdCodingStyeFeature.cs Outdated
Comment thread core/src/Baked/CodingStyle/Id/IdCodingStyeFeature.cs Outdated
Comment thread core/src/Baked/CodingStyle/Id/IdCodingStyeFeature.cs Outdated
dncsvr added 4 commits January 20, 2026 17:05
@dncsvr
review changes
4ba7f88 
- move global usings to their own template file
@dncsvr
review changes
42c844a 
- refactor IdMapperTemplate into single file
@dncsvr
review changes
93337f4 
- fix format
@dncsvr
fix indentation
cf1fbf7
@dncsvr dncsvr merged commit 762e49d into mouseless:epic/locatability Jan 20, 2026
9 checks passed
@dncsvr dncsvr deleted the issue/locatability/id-configuration-as-feature branch January 20, 2026 14:49
@dncsvr dncsvr mentioned this pull request Jan 20, 2026
Merged
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cihandeniz cihandeniz cihandeniz approved these changes

Assignees

@dncsvr dncsvr

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@dncsvr @cihandeniz