Skip to content

Task output file download link reference - #228

Merged
julien-nc merged 5 commits into
mainfrom
enh/noid/task-output-file-reference
Apr 29, 2025
Merged

Task output file download link reference#228
julien-nc merged 5 commits into
mainfrom
enh/noid/task-output-file-reference

Conversation

@julien-nc

@julien-ncjulien-nc commented Apr 28, 2025

Copy link
Copy Markdown
Member

2 things:

Remaining issue: The Viewer is not displayed on top of the assistant modal. Everything is fine if we open the viewer from a link widget in Talk, or from the assistant standalone page, but no luck if the assistant itself is already in a modal.
Solution: Change the NcModal container prop to #content. The Viewer is then displayed on top.

Kooha-2025-04-28-11-38-41.webm

@julien-nc

Copy link
Copy Markdown
MemberAuthor

cc @jancborchardt

@janepiejanepie left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, I like it!

@marcelklehrmarcelklehr left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Really cool!

Comment threadlib/Controller/AssistantApiController.php Outdated
Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
… a save button in MediaField
Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
…hen displayed on top
Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
@julien-nc
julien-ncforce-pushed the enh/noid/task-output-file-reference branch from e543fb5 to 7800142CompareApril 29, 2025 14:24
@julien-nc
julien-nc merged commit ccda174 into mainApr 29, 2025
@julien-nc
julien-nc deleted the enh/noid/task-output-file-reference branch April 29, 2025 14:26
@julien-ncjulien-nc mentioned this pull request Jun 26, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewenhancementNew feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@julien-nc@marcelklehr@edward-ly@janepie