Part of #2285413: [Meta] Standardize entity route names. See there for details.

CommentFileSizeAuthor
#2 route-names-configtest.patch4.36 KBCrell
route-names-configtest.patch5.07 KBCrell
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Needs review » Needs work 
+++ b/core/modules/config/tests/config_test/config_test.links.action.yml
@@ -1,5 +1,5 @@
-  route_name: config_test.entity_add
+  route_name: entity.config_test.edit_form_add

+++ b/core/modules/config/tests/config_test/config_test.routing.yml
@@ -6,14 +6,14 @@ config_test.list_page:
-config_test.entity_add:
+entity.config_test.edit_form_add:

i am sorry but this route name cannot be right. Just go with add-form/add_form

Crell’s picture

Crell CreditAttribution: Crell commented
Status: Needs work » Needs review
FileSize
route-names-configtest.patch4.36 KB

Meh. Undid that one.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Needs review » Reviewed & tested by the community

Fine.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Status: Reviewed & tested by the community » Fixed

Committed f3cda09 and pushed to 8.0.x. Thanks!

  • alexpott committed f3cda09 on 8.0.x 
    Issue #2314859 by Crell: Standardize ConfigTest route names.

Status: Fixed » Closed (fixed)

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