s_leu โ changed the visibility of the branch 2546212-entity-viewform-mode-10.3.x to hidden.
- First commit to issue fork.
- heddn Nicaragua
I think I fixed the last failing test. As far as the 403 access denied when there isn't any config translations to translate as reported in #289 and elsewhere, I think that is an external issue to this.
ConfigTranslationOverviewAccess::doCheckAccess
has a check for$mapper->hasTranslatable()
. If there are no translations, then you get a 403. But that is an existing issue that is only exacerbated by what we're doing here.See screenshot of what it looks like if you get rid of that check:
There are deeper issues to solve around UX with a blank translation form if we remove that check. Which I think deserves its own dedicated issue to discuss. But that shouldn't hold this up here.
- heddn Nicaragua
I grabbed the diffs for 8984 & 4294. It looks like some cruft introduced itself into 8984 so I've hidden it. Going to take 4294 out for a ride.