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.

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

[2.0.1] RevEng: Fix scaffolding for nvarchar(4000) #10004

Merged
merged 1 commit into from
Oct 8, 2017
Merged

Conversation

smitpatel
Copy link
Contributor

Types which requires length specified returns needs default length of 8000 specified

Resolves #9963

@smitpatel smitpatel changed the base branch from dev to rel/2.0.1 October 6, 2017 23:30
@smitpatel smitpatel closed this Oct 6, 2017
@smitpatel smitpatel reopened this Oct 6, 2017
@@ -239,7 +239,8 @@ protected override void OnModelCreating(ModelBuilder modelBuilder)

entity.Property(e => e.TypeAliasColumn)
.HasColumnName("typeAliasColumn")
.HasColumnType("TestTypeAlias");
.HasColumnType("TestTypeAlias")
.HasMaxLength(4000);
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Filed #10003

Types which requires length specified returns needs default length of 8000 specified

# Conflicts:
#	test/EFCore.SqlServer.FunctionalTests/Scaffolding/SqlServerDatabaseModelFactoryTest.cs
@smitpatel smitpatel merged commit 196400e into rel/2.0.1 Oct 8, 2017
@smitpatel smitpatel deleted the port9188 branch October 8, 2017 00:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants