- Issue created by @rominronin
- 🇦🇹Austria rominronin Vienna
A publicly accessible image URL needs to be delivered to the API. This is not possible in local development or protected stage environments (without additional work).
Finding a solution to this is currently beyond the scope of the module.
We decided to notify users of API errors using Drupal's form field validation messages, so they at least understand why alt text cannot be generated (in non-public environments).
- 🇦🇹Austria rominronin Vienna
The validation message was included. There is however another issue: the functionality doesn't work on the node/add form at all. This needs reviewing and fixing.
- 🇦🇹Austria rominronin Vienna
I took a deeper dive into what's going on behind the image alt text issue, and there are numerous issues with the workflow and implementation, so I think it's better to redefine what's intended and work towards that:
- As normal, a site builder can define, per image field, if alt or title fields should be enabled
- Sidekick can be enabled on the form display settings page per image field.
- IF enabled, there should be a checkbox to select alt field (as the sidekick target for this field)
- On the form (add or edit) page, the 'enable sidekick' setting should be checked, then the generate button should be displayed according to the form field settings.
- If 'enable sidekick' is not checked, the default core settings should be respected without interferenceThis last point is just one of the issues we currently have with the workflow.
- 🇦🇹Austria rominronin Vienna
Disregard the last message for now (possibly turn it into a future task).
While debugging the issue I found the cause, and it will require a separate task to fix completely 3485590 🐛 sidekick.js conflicts Active .
For now, I implemented a fix for the alt text generation on create forms, bear in mind if you select any of the `Show Sidekick in Sidebar` options except `Don't display` (admin/config/services/sidekick), you will likely encounter issues until that task is resolved.
Marking as fixed for now.
Automatically closed - issue fixed for 2 weeks with no activity.