Entities have advanced methods to define their routes rather than enumerating them in the *.routing.yml file.

CommentFileSizeAuthor
#5 interdiff.txt696 bytespfrenssen
#5 3111569-5.patch3.38 KBpfrenssen
#2 3111569-2.patch2.7 KBclaudiu.cristea
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

claudiu.cristea created an issue. See original summary.

claudiu.cristea’s picture

claudiu.cristea
Primary language Romanian
Location Arad 🇷🇴
CreditAttribution: claudiu.cristea as a volunteer and at Webikon for European Commission and European Union Institutions, Agencies and Bodies commented
Status: Active » Needs review
FileSize
3111569-2.patch2.7 KB

Patch

Status: Needs review » Needs work

The last submitted patch, 2: 3111569-2.patch, failed testing. View results

pfrenssen’s picture

pfrenssen
Location Sofia
CreditAttribution: pfrenssen at Randstad Digital for European Commission and European Union Institutions, Agencies and Bodies commented

This is looking good but we need to update the tests to use the improved entity type label.

pfrenssen’s picture

pfrenssen
Location Sofia
CreditAttribution: pfrenssen at Randstad Digital for European Commission and European Union Institutions, Agencies and Bodies commented
Status: Needs work » Needs review
FileSize
3111569-5.patch3.38 KB
interdiff.txt696 bytes

Updated tests.

pfrenssen’s picture

pfrenssen
Location Sofia
CreditAttribution: pfrenssen at Randstad Digital for European Commission and European Union Institutions, Agencies and Bodies commented

Tests are now passing on Drupal 8 and Drupal 9, merging in the develop branch. Thanks for the contribution!

pfrenssen’s picture

pfrenssen
Location Sofia
CreditAttribution: pfrenssen at Randstad Digital for European Commission and European Union Institutions, Agencies and Bodies commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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