Uh oh!
There was an error while loading. Please reload this page.
feat: files "new file" menu generate image action - #322
Conversation
60a7b76 to
002945eComparejulien-nc
commented
Jul 30, 2025
Hey, nicely done. I rebased on main, fixed the conflicts and merged the 2 listeners for |
julien-nc
left a comment
There was a problem hiding this comment.
Looks good!
Could you use a different listener than TaskSuccessfulListener like NewFileMenuTaskSuccessfulListener? It would make it easier to read and the logic is kind of separated anyway.
We don't care about oc_assistant_task_notif for the new file menu tasks, we always send a notification.
| 'openfile' => 'false', | ||
| ], | ||
| ); | ||
| } catch (\Exception $e) { |
There was a problem hiding this comment.
We need a failure notification in this case.
There was a problem hiding this comment.
Should I also create another TaskFailedEvent listener (to not mix logic with existing) to produce task failure notification?
There was a problem hiding this comment.
I think so, yes. It would be nicer to have a new type of notification for this case. Something like "The new image generation in FOLDER_RICH_OBJECT has failed" and an action button (label "View task") to open the assistant on the task.
There was a problem hiding this comment.
The success notification can stay like it is now IMO.
Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
… openAssistantTask Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
002945e to
03696cfCompareSigned-off-by: Julien Veyssier <julien-nc@posteo.net>
julien-nc
commented
Aug 5, 2025
@andrey18106 I hope you don't mind, I made the changes i suggested 😁 :
|
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
Resolves: #305
This PR adds new entry "Generate image" to "New file" menu in Files app.
Custom dialog is opened to enter prompt and schedule
core:text2imagetask to generate one image.TaskSuccessfulEventlistener is adjusted to handle this task withcustomIdset tonew-image-file:<folderId>to store the result and create notification with the target link to new image file.This task also kept visible in the Assistant UI history.
It is possible to disable it with
occ config:app:set assistant disableFilesNewMenuPlugin --value 1.