- Issue created by @jlbellido
- Merge request !22#3513376: Move help text to hook_help() instead of hook_install. β (Merged) created by jlbellido
- πͺπΈSpain jlbellido
I'm sending a first MR with the suggested approach which works for me.
- πΊπΈUnited States itmaybejj
Questions --
- There's no route named editoria.settings in the module repo -- is that a typo here in the queue, or is it possible there's a patch/typo in your dev environment?
- It looks like this fix requires the Help module to be both installed and configured? I'd rather not add that dependency if possible.
- I'm having trouble reproducing this. Is the install failing on your end, or is the install finishing and some local test suite is running before a cache clear and throwing errors in the console?
-
itmaybejj β
committed e1e82181 on editoria11y-3513376-3513376-installation-fails
Issue #3513376 by jlbellido, itmaybejj: Installation fails due to route...
-
itmaybejj β
committed e1e82181 on editoria11y-3513376-3513376-installation-fails
-
itmaybejj β
committed fdf7c1c1 on editoria11y-3513376-3513376-installation-fails
Issue #3513376 by jlbellido, itmaybejj: Installation fails due to route...
-
itmaybejj β
committed fdf7c1c1 on editoria11y-3513376-3513376-installation-fails
-
itmaybejj β
committed 0b258e3f on editoria11y-3513376-3513376-installation-fails
Issue #3513376 by jlbellido, itmaybejj: Installation fails due to route...
-
itmaybejj β
committed 0b258e3f on editoria11y-3513376-3513376-installation-fails
- πΊπΈUnited States itmaybejj
Meanwhile -- see if that change works for you? It moves the message from the Help hook to the post_update hook, which only fires after the module is installed and the route exists. I hope...
-
itmaybejj β
committed 27e074a8 on 2.2.x authored by
jlbellido β
git commit -m 'Issue #3513376 by itmaybejj, jlbellido: Installation...
-
itmaybejj β
committed 27e074a8 on 2.2.x authored by
jlbellido β