-
Notifications
You must be signed in to change notification settings - Fork 23
feat: onedrive word write tool #413
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
36dd4b6
to
03af393
Compare
word/credential/tool.gpt
Outdated
@@ -16,6 +16,8 @@ Tools: ../../oauth2 | |||
"Fields.Read", |
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.
I think you should replace Fields.Read
to File.Read
. I think `"Fields.Read" are added accidentlly as a typo. The scope doesn't exist
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.
Yes, this was my fault.
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.
I guess they could be removed as we have ReadWrite below, but I'll leave them here as the Azure permissions played tricks on me more than once :D
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.
This looks good to me once the invalid Fields
scopes are removed.
Co-authored-by: Grant Linville <grant@acorn.io>
Co-authored-by: Grant Linville <grant@acorn.io>
Co-authored-by: Grant Linville <grant@acorn.io>
Co-authored-by: Grant Linville <grant@acorn.io>
f82360b
to
94a1427
Compare
Adds a new
Write Doc
tool that allows for creating new docx documents or updating existing ones in OneDrive.It works with OneDrive item IDs or filepaths within OneDrive, creating directories as needed.
Conversion from Markdown to Docx is done either via Pandoc or LibreOffice, whichever is available.
In our docker images we definitely have LibreOffice (soffice binary) available.
Example of a Doc after:
Issue obot-platform/obot#1468
Note: It may make sense to also rely on pandoc/soffice instead of docconv to transform back from docx to markdown when reading a file.