-
Notifications
You must be signed in to change notification settings - Fork 17
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
base: main
Are you sure you want to change the base?
Conversation
acd0ffd
to
b5cb9eb
Compare
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-dev For the capabilities to be declared by the assistant, you need some AI providers. There are some fake ones in the |
Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
…add multi filters, add method Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
944ea1e
to
e4f9dc8
Compare
…n file Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
e4f9dc8
to
7d3d17a
Compare
closes #340
Add declarative UI capabilities for clients.
Add one hook for each of our file actions (summarize, stt and tts).
Currently unknown: