This repository has been archived by the owner on Feb 23, 2023. It is now read-only.

Readme should suggest text instead of varchar #28

Open

heartdriven opened this issue Nov 3, 2017 · 0 comments

heartdriven commented Nov 3, 2017

In step 6 of the installation instructions in the readme file there's mentioned:

Create a varchar (string) column in a table on your database to match the config file in step 5.

.. but in fact a varchar is too short (255 chars) and text() should be used instead of string(), just like in the generated migration.

Grimthorr changed the title ~~Misleading instructions in readme~~ Readme should suggest text instead of varchar

ricardoboss added a commit to ricardoboss/laravel-user-settings that referenced this issue


          Address Grimthorr#28

6ec218c

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.