We should add schema for field storage, instance, widget, and formatter.

CommentFileSizeAuthor
#5 3077525-5.patch1.1 KBhussainweb
#4 do_username-3077525-4.patch1.07 KBjunkuncz
#3 do_username-3077525-3.patch1.08 KBjunkuncz

Comments

hussainweb created an issue. See original summary.

junkuncz’s picture

junkuncz
Primary language Hungarian
Location Budapest
commented
Assigned: Unassigned » junkuncz

I will have a look soon!

junkuncz’s picture

junkuncz
Primary language Hungarian
Location Budapest
commented
Assigned: junkuncz » Unassigned
Status: Active » Needs review
Related issues: +#3077524: Add more formatting options relevant to the link
StatusFileSize
new do_username-3077525-3.patch1.08 KB

Patch is attached!

We need to review and merge the attached patch from this issue first: https://www.drupal.org/project/do_username/issues/3077524
Otherwise the schema of the formatter will not be fine.

junkuncz’s picture

junkuncz
Primary language Hungarian
Location Budapest
commented
StatusFileSize
new do_username-3077525-4.patch1.07 KB

New patch has been attached (had an issue with the first one).

hussainweb’s picture

hussainweb
Primary language English
commented
StatusFileSize
new 3077525-5.patch1.1 KB

Updating the schema after #3143656: Remove unnecessary field storage settings and fixing other errors. The schema config for storage was in field settings in the previous patch.

  • hussainweb committed 83b5ba9 on 8.x-1.x authored by junkuncz 
    Issue #3077525 by junkuncz, hussainweb: Add schema for settings
hussainweb’s picture

hussainweb
Primary language English
commented

Thanks for the schema. It was very helpful in identifying hidden issues with the schema along with the tests in #3143632: Add a config schema test

hussainweb’s picture

hussainweb
Primary language English
commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.