Problem/Motivation

One of the test modules has incorrect routing information which breaks when config_translation is enabled. Found whilst working on #2762273: Config import issue with LingotekConfigSubscriber

Proposed resolution

Fix it.

Remaining tasks

User interface changes

None

API changes

None - test code.

Data model changes

None

CommentFileSizeAuthor
#2 2762347-2.patch641 bytesalexpott

Comments

alexpott created an issue. See original summary.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Active » Needs review
StatusFileSize
new 2762347-2.patch641 bytes
alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Related issues: +#2762273: Config import issue with LingotekConfigSubscriber
penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾 Seville 💃 Andalusia, UTC+2 🇪🇺
commented
Status: Needs review » Reviewed & tested by the community

Great catch!

I don't think we need tests for a bug in tests... so it's RTBC

penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾 Seville 💃 Andalusia, UTC+2 🇪🇺
commented
Issue tags: +Quick fix

  • catch committed 962c990 on 8.2.x 
    Issue #2762347: config_test_no_status has wrong edit route

  • catch committed 4d9e857 on 8.1.x 
    Issue #2762347: config_test_no_status has wrong edit route

(cherry...
catch’s picture

catch
he/him
Primary language English
commented

Committed/pushed to 8.2.x and cherry-picked to 8.1.x. Thanks!

catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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