Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Enforce class and new() for Strong IDs #220

Closed
bfren opened this issue Mar 27, 2022 · 0 comments · Fixed by #223
Closed

Enforce class and new() for Strong IDs #220

bfren opened this issue Mar 27, 2022 · 0 comments · Fixed by #223
Assignees
@bfren
Milestone
v8.0.0

Comments

@bfren
Copy link
Owner

bfren commented Mar 27, 2022 •
edited
Loading

MVC model binding requires classes (not structs) and parameterless constructors
@bfren bfren added this to the v8.0.0 milestone Mar 27, 2022
@bfren bfren self-assigned this Mar 27, 2022
@bfren bfren changed the title Make WP Id classes structs Enforce class and new() for Strong IDs Mar 27, 2022
bfren added a commit that referenced this issue Mar 27, 2022
@bfren
Making StrongIds structs - #220
867c163
bfren added a commit that referenced this issue Mar 27, 2022
@bfren
Don't initialise IDs with default values - #220
3bd48a4
bfren added a commit that referenced this issue Mar 27, 2022
@bfren
Use struct pattern matching for Ids - #220
1c33c30
bfren added a commit that referenced this issue Mar 27, 2022
@bfren
Adding class, new() constraints to IWithId - #220
069ec88
bfren added a commit that referenced this issue Mar 27, 2022
@bfren
Updating constraints, using parameterless constructors - #220
66147c8
@bfren bfren linked a pull request Mar 27, 2022 that will close this issue
Enforce class and new() for Strong IDs (#220) #223
Merged
bfren added a commit that referenced this issue Mar 27, 2022
@bfren
Merge pull request #223 from bfren/220-make-wp-id-classes-structs
115192d 
Enforce class and new() for Strong IDs (#220)
@bfren bfren closed this as completed Mar 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bfren bfren

Labels
None yet
Projects
None yet
Milestone
v8.0.0
Development

Successfully merging a pull request may close this issue.

Enforce class and new() for Strong IDs (#220) bfren/jeebs
1 participant
@bfren