Field group has no D10 support yet, this issue will remove the tests first, then bring back when field group has code (in dev) with D10 support.

Note, only in 5.0.x branch, don't bother fixing that in the 8.x-3.x one.

CommentFileSizeAuthor
#9 3308854-9.patch6.08 KBswentel

Comments

swentel created an issue. See original summary.

  • swentel committed 6873baa on 5.0.x
    Issue #3308854: turn off fieldgroup in tests
    

  • swentel committed cfadbeb on 5.0.x
    Issue #3308854: turn off fieldgroup in tests
    

  • swentel committed 45569d3 on 5.0.x
    Issue #3308854: turn off fieldgroup in tests
    

  • swentel committed 5b761c9 on 5.0.x
    Issue #3308854: turn off fieldgroup in tests
    
swentel’s picture

Issue summary: View changes

  • swentel committed dd12beb on 5.0.x
    Issue #3308854: turn off field group in tests
    
swentel’s picture

Priority: Normal » Critical

#3278537: D10 compatibility | Declaration of Drupal\field_group\Routing\RouteSubscriber::getSubscribedEvents() must be compatible with Drupal\Core\Routing\RouteSubscriberBase::getSubscribedEvents() is in, so we can start testing field group with DS and add the tests back.

Marking critical because field group support is probably broken completely

swentel’s picture

Status: Active » Needs review
Related issues: +#3120298: Order of alter hooks is not always respected by ModuleHandler
StatusFileSize
new6.08 KB

This works, however, might be bitten by #3120298: Order of alter hooks is not always respected by ModuleHandler.

So even if the tests are green now (probably after commit), let's hope that the module implements thing is just something weird in the test only :)

  • swentel committed 5c7ae26 on 5.0.x
    Issue #3308854 by swentel: Field group support
    
swentel’s picture

Priority: Critical » Normal
Status: Needs review » Fixed

committed, hoping that the module order is something weird in the tests, so leaving for now though.

swentel’s picture

Status: Fixed » Closed (fixed)