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

Update create_phrases_table.php #55

Merged
merged 1 commit into from
Jan 22, 2024
Merged

Conversation

NoahNxT
Copy link
Contributor

@NoahNxT NoahNxT commented Jan 22, 2024

Solves the issue on the installation of running the migrations. Reference issue: #53

In Connection.php line 822:

SQLSTATE[42000]: Syntax error or access violation: 1101 BLOB, TEXT, GEOMETRY or JSON column 'status' can't have a default value (Connection: mysql, SQL: create table ltu_phrases (id bigint unsigned not null auto_increment primary key, uuid char(36)
not null, translation_id bigint unsigned not null, translation_file_id bigint unsigned not null, phrase_id bigint unsigned null, key varchar(255) not null, group varchar(255) not null, value text null, status longtext not null default 'activ
e', parameters json null, note text null, created_at timestamp null, updated_at timestamp null) default character set utf8mb4 collate 'utf8mb4_unicode_ci')

In Connection.php line 574:

SQLSTATE[42000]: Syntax error or access violation: 1101 BLOB, TEXT, GEOMETRY or JSON column 'status' can't have a default value

@MohmmedAshraf MohmmedAshraf merged commit 7f8755a into MohmmedAshraf:main Jan 22, 2024
5 checks passed
@MohmmedAshraf
Copy link
Owner

Thanks @NoahNxT

@NoahNxT
Copy link
Contributor Author

NoahNxT commented Jan 22, 2024

Don't forget to update and close #53 💪 Keep up the good work! We already integrated it at our company and we are very happy with it 🫡

@MohmmedAshraf
Copy link
Owner

Glad you're enjoying the package! 😍, The status field wasn't intended to be there; it was initially planned for upcoming features related to versions and history. Unfortunately, I forgot to remove it before the release, since you've fixed the error, I'll keep it for now and consider you a new contributor. 😃 Thanks

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.

2 participants