UX improvements for site builders using the AI Automators
Merge request reports
Activity
15 * Checks if access to advanced AI Automator features should be granted. 16 */ 17 final class AutomatorAdvancedAccessChecker implements AccessInterface { 18 19 /** 20 * Constructs an AutomatorAdvancedAccessChecker object. 21 */ 22 public function __construct( 23 private readonly ModuleHandlerInterface $moduleHandler, 24 ) {} 25 26 /** 27 * Access callback. 28 */ 29 public function access(Route $route, AccountInterface $account): AccessResult { 30 if ($this->moduleHandler->moduleExists('ai_ckeditor') || Settings::get('ai_automator_advanced_mode_enabled', FALSE)) { -
This is great! The only comment on this row and in the install text under:
I think its enough with the settings here - most people that install the AI CKEditor will just use the default built in plugins, that doesn't use Automators. Since we will add more "consumers" of the custom entities, its fine that this is purely for developers at the moment and a problem how we move to builder once we have more modules using it.
No, I don't think that's a good idea: it makes assumptions about how other people want to use the module and not all of them will have the ability to update the settings.php. I agree that all the overhead of the AI CK Editor Integration/AI Automator Chains stuff is going to be too much for most users, but if we're not going to move it into its own module that they can enable or disable as they wish I don't think the solution is to hide it when it's physically possible that they can use it.
added 6 commits
- ce3783da - Update README with accurate description and dependencies. #3491583
- c04d219d - Delete obsolete and incomplete documentation refering to previous version of this module. #3491583
- e19555d4 - Fix broken path in mkdocs.yml. #3491583
- 0c254293 - Move Automator Chain Type paths to be under the AI Core parent menu so menu...
- 812ed1fb - Document the use of the AI Automators module. #3491583
- 91b771b4 - Fix typo in pth and move sub-module instructions into the correct menu. #3491583
Toggle commit listadded 1 commit