- Issue created by @phenaproxima
- πΊπΈUnited States phenaproxima Massachusetts
It might be worth postponing this on β¨ If a module installs additional dependencies, they should show up as installed right away Active , since that will remove the inline project refresh that necessitates a weird dependency injection-related workaround.
- πΊπΈUnited States phenaproxima Massachusetts
Postponing on β¨ If a module installs additional dependencies, they should show up as installed right away Active , which is already reviewable.
- πΊπΈUnited States phenaproxima Massachusetts
This is gonna need explicit test coverage.
- πΊπΈUnited States phenaproxima Massachusetts
Also postponing on π RecipeForm doesn't set a checkpoint Active , since the logic change there will affect this issue.
- πΊπΈUnited States phenaproxima Massachusetts
And there goes the other blocker.
- πΊπΈUnited States phenaproxima Massachusetts
Assigning to the guy who will manually test and hopefully commit this bad boy. :) I still need to do a second MR to backport to 2.0.x.
- πΊπΈUnited States chrisfromredfin Portland, Maine
Found an issue that if you apply three recipes and only two require config, it gives you the wrong titles on the fieldsets in the modal. To replicate, select article tags, AI assistant, and website contact form. (I'm not sure if order matters.) Test may need to be either improved or written.
- πΊπΈUnited States phenaproxima Massachusetts
Nice catch. Fixed the bug and adjusted the tests. Turns out that order of selection was significant.
-
chrisfromredfin β
committed 9bc5a9cf on 2.1.x authored by
phenaproxima β
Issue #3539152: When multiple recipes with inputs are activated, show...
-
chrisfromredfin β
committed 9bc5a9cf on 2.1.x authored by
phenaproxima β
-
chrisfromredfin β
committed c0bf2047 on 2.0.x authored by
phenaproxima β
Issue #3539152: When multiple recipes with inputs are activated, show...
-
chrisfromredfin β
committed c0bf2047 on 2.0.x authored by
phenaproxima β