The Needs Review Queue Bot → tested this issue. It either no longer applies to Drupal core, or fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".
Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.
Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.
- last update
8 months ago Patch Failed to Apply - 🇷🇴Romania amateescu
I just closed 2 issues in the IP address fields → module which show that this is a real problem that needs to be fixed in core:
- 🐛 Unable to execute related tests via sqlite successfully Active
- 🐛 Error when the project is over postresql Closed: duplicate - Merge request !10415Issue #3118014: Additional blob field sizes in schema definitions → (Open) created by gapple
- 🇮🇳India arunsahijpal
The pipeline is all green now and the MR seems to be working fine
Hence moving it to RTBC. - heddn Nicaragua
+1 on RTBC. I don't think additional test coverage is needed. LGTM.
- 🇺🇸United States smustgrave
Postponed 🐛 layout_builder__layout_section column hitting database limit Needs work on this one.
- 🇳🇱Netherlands daffie
I think we should add testing for this change. Lets add testing for:
- as a field of a new table
- add a field to an existing table
- change an existing field to use the new field typeFor each way to create a field add a check that the field is of the correct type and do that for each new blob type.
Please add a change record
Changed the issue to a feature request.