starred/odin-maelstrom-cms

Fork 0

mirror of https://github.com/maelstrom-cms/odin.git synced 2026-04-25 08:16:03 +03:00

[GH-ISSUE #10] Can not save open graph #5

New issue

Closed

opened 2026-02-27 09:12:32 +03:00 by kerem · 5 comments

kerem commented

2026-02-27 09:12:32 +03:00

Owner

Originally created by @nalakawula on GitHub (Oct 8, 2019).
Original GitHub issue: https://github.com/maelstrom-cms/odin/issues/10

Here the log:

[2019-10-08 13:12:50] production.ERROR: SQLSTATE[22001]: String data, right truncated: 1406 Data too long for column 'description' at row 1 (SQL: insert into `open_graph_scans` (`title`, `description`, `image`, `url`, `icon`, `website_id`, `updated_at`, `created_at`) values
...

Thank you

Originally created by @nalakawula on GitHub (Oct 8, 2019). Original GitHub issue: https://github.com/maelstrom-cms/odin/issues/10 Here the log: ``` [2019-10-08 13:12:50] production.ERROR: SQLSTATE[22001]: String data, right truncated: 1406 Data too long for column 'description' at row 1 (SQL: insert into `open_graph_scans` (`title`, `description`, `image`, `url`, `icon`, `website_id`, `updated_at`, `created_at`) values ... ``` Thank you

kerem closed this issue

2026-02-27 09:12:32 +03:00

kerem commented

2026-02-27 09:12:33 +03:00

Author

Owner

@OwenMelbz commented on GitHub (Oct 8, 2019):

How long is your description?

@OwenMelbz commented on GitHub (Oct 8, 2019): How long is your description?

kerem commented

2026-02-27 09:12:33 +03:00

Author

Owner

@nalakawula commented on GitHub (Oct 8, 2019):

371 long.
I check your description column is varchar. So i just need to update into text type right?

@nalakawula commented on GitHub (Oct 8, 2019): 371 long. I check your description column is varchar. So i just need to update into text type right?

kerem commented

2026-02-27 09:12:33 +03:00

Author

Owner

@OwenMelbz commented on GitHub (Oct 8, 2019):

I would checkout -> https://blog.spotibo.com/meta-description-length/

That length is far too long for the description, which is why this is throwing an error.

But yes changing it to text will allow for longer descriptions

@OwenMelbz commented on GitHub (Oct 8, 2019): I would checkout -> https://blog.spotibo.com/meta-description-length/ That length is far too long for the description, which is why this is throwing an error. But yes changing it to `text` will allow for longer descriptions

kerem commented

2026-02-27 09:12:33 +03:00

Author

Owner

@nalakawula commented on GitHub (Oct 8, 2019):

nice best practice advise.
I will close this issue.

Thank you.

@nalakawula commented on GitHub (Oct 8, 2019): nice best practice advise. I will close this issue. Thank you.

kerem commented

2026-02-27 09:12:33 +03:00

Author

Owner

@OwenMelbz commented on GitHub (Oct 8, 2019):

I've added a migration which does cater for longer descriptions so it does not throw an error.

But has given me the idea for maybe a warning on the UI saying description/title is too long!

@OwenMelbz commented on GitHub (Oct 8, 2019): I've added a [migration](https://github.com/maelstrom-cms/odin/commit/e9f7f6e37544ed80233950fa98db49970abf01a4) which does cater for longer descriptions so it does not throw an error. But has given me the idea for maybe a warning on the UI saying description/title is too long!

kerem referenced this issue

2026-02-27 09:12:43 +03:00

[PR #5] [MERGED] Add DOCTYPE to maelstrom::layouts.wrapper #39