skttl / umbraco-fulltextsearch8

Full Text indexing and searching for Umbraco 8 and Examine.
MIT License
19 stars 22 forks source link

Consider changing SQL server field type from ntext to nvarchar(max)? #94

Closed hfloyd closed 1 year ago

hfloyd commented 1 year ago

As per Microsoft:

The ntext, text, and image data types will be removed in a future version of SQL Server. Avoid using these data types in new development work, and plan to modify applications that currently use them. Use nvarchar(max), varchar(max), and varbinary(max) instead.

Should the SQL table be updated here?

skttl commented 1 year ago

If Microsoft says so, we better adjust :)

Great candidate for a new major too!

skttl commented 1 year ago

Fixed in d851e7f :)