-
Notifications
You must be signed in to change notification settings - Fork 26
feat(declarativeui): add hooks for each of our file actions #344
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
acd0ffd to
b5cb9eb
Compare
marcelklehr
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks reasonable
a5b2673 to
72e2f65
Compare
|
When combining the capabilities declared by various apps, the server combines the arrays. For objects, all good. For lists, as the keys are preserved and a Php list is almost an object with integer keys, the combining goes wrong and each list overwrites the other ones. Here is my suggestion for the
Object in the {
"assistant": {
"context-menu": [
{
"name": "Summarize",
"url": "/ocs/v2.php/apps/assistant/api/v1/file-action/{fileId}/core:text2text:summary",
"method": "POST",
"mimetype_filters": "text/, application/msword, application/vnd.openxmlformats-officedocument.wordprocessingml.document, application/vnd.oasis.opendocument.text, application/pdf",
"icon": "/apps-extra/assistant/img/declarativeui/summarize.svg"
},
{
"name": "Transcribe audio",
"url": "/ocs/v2.php/apps/assistant/api/v1/file-action/{fileId}/core:audio2text",
"method": "POST",
"mimetype_filters": "audio/",
"icon": "/apps-extra/assistant/img/declarativeui/speech_to_text.svg"
},
{
"name": "Text to speech",
"url": "/ocs/v2.php/apps/assistant/api/v1/file-action/{fileId}/core:text2speech",
"method": "POST",
"mimetype_filters": "text/, application/msword, application/vnd.openxmlformats-officedocument.wordprocessingml.document, application/vnd.oasis.opendocument.text, application/pdf",
"icon": "/apps-extra/assistant/img/declarativeui/text_to_speech.svg"
}
]
},
"cospend": {
"context-menu": [
{
"name": "cospend fake",
"url": "hello"
}
]
}
} |
|
@tobiasKaminsky You can test this branch by checking it out and building it: npm ci
npm run dev
composer i --no-devFor the capabilities to be declared by the assistant, you need some AI providers. There are some fake ones in the |
e4f9dc8 to
7d3d17a
Compare
3f9001a to
5450fa6
Compare
2dd3bc9 to
25154f2
Compare
Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
…add multi filters, add method Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
…n file Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
version 0.1 Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
0e0797c to
211a03d
Compare
Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
211a03d to
c9d4fb8
Compare
closes #340
Add declarative UI capabilities for clients.
Add one hook for each of our file actions (summarize, stt and tts).
Currently unknown: