There are an issue on schemas and settings for segment_display_limit

1 error(s) identified by config_inspector to fix :
 ------------------------------------------------ ---------------- 
  Key                                              Status          
 ------------------------------------------------ ---------------- 
  easy_breadcrumb.settings:segment_display_limit   missing schema  
 ------------------------------------------------ ----------------
CommentFileSizeAuthor
#4 3259819-4.patch959 bytesvacho
#3 3259819-3.patch473 bytesvacho

Issue fork easy_breadcrumb-3259819

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

vacho created an issue. See original summary.

vacho opened merge request !35

vacho’s picture

vacho CreditAttribution: vacho at Skilld commented
FileSize
3259819-3.patch473 bytes

Patch to fix.

vacho’s picture

vacho CreditAttribution: vacho at Skilld commented
FileSize
3259819-4.patch959 bytes

Fixed one more issue at schema.

renatog’s picture

renatog
Location Campinas
CreditAttribution: renatog at CI&T commented
Status: Active » Reviewed & tested by the community

Yeah, I applied the fix and it really makes sense. segment_display_limit with 0 instead of an empty string '' and segment_display_limit using "integer" instead of "number". Good catch. +1 to this

  • RenatoG committed efd1bd0 on 2.x authored by vacho 
    Issue #3259819 by vacho, RenatoG: Missing schema for...
renatog’s picture

renatog
Location Campinas
CreditAttribution: renatog at CI&T commented
Status: Reviewed & tested by the community » Fixed

Moved to the dev branch

Thank you so much for your contribution

Status: Fixed » Closed (fixed)

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