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

DateTime Required Annotation #207

Open
geezz07 opened this issue Apr 1, 2022 · 1 comment
Open

DateTime Required Annotation #207

geezz07 opened this issue Apr 1, 2022 · 1 comment
Labels
bug Something isn't working

Comments

@geezz07
Copy link

geezz07 commented Apr 1, 2022

Hi there,

In my database I have a column for a DateTime that is required however the "Required" data annotation is not appearing. Should the line in the code below be "!property.ClrType.IsNullableType()"?

EntityFrameworkCore.Scaffolding.Handlebars/src/EntityFrameworkCore.Scaffolding.Handlebars/HbsCSharpEntityTypeGenerator.cs

Line 630 in 2b65b77

&& property.ClrType.IsNullableType()

Thanks
@tonysneed
Copy link
Contributor

I'll see if I can reproduce.

If you like, could you create a Pull Request simple unit test?

@tonysneed tonysneed added the bug Something isn't working label May 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tonysneed @geezz07