Feat: add mail inbox tools #54
Draft
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
marcelklehr
reviewed
Jul 28, 2025
ex_app/lib/all_tools/mail.py
Outdated
| @safe_tool | ||
| def get_mail_folder_list(account_id: int): | ||
| """ | ||
| Lists all mail folders account. You need to get the correct account id matching the request first before using this tool. |
There was a problem hiding this comment.
Suggested change
| Lists all mail folders account. You need to get the correct account id matching the request first before using this tool. | |
| Lists all mail folders for an account. You need to get the correct account id matching the request first before using this tool. |
marcelklehr
reviewed
Jul 28, 2025
ex_app/lib/all_tools/mail.py
Outdated
| def list_mails(folder_id: int, n_mails: int = 30): | ||
| """ | ||
| Lists all messages in a mailbox folder. You need to get the correct folder id matching the request first before using this tool. | ||
| :param folder_id: The id of the folder to list as integer, obtainable via folder |
There was a problem hiding this comment.
Suggested change
| :param folder_id: The id of the folder to list as integer, obtainable via folder | |
| :param folder_id: The id of the folder to list as integer, obtainable via get_mail_folder_list |
marcelklehr
reviewed
Jul 28, 2025
ex_app/lib/all_tools/mail.py
Outdated
| """ | ||
| Lists all messages in a mailbox folder. You need to get the correct folder id matching the request first before using this tool. | ||
| :param folder_id: The id of the folder to list as integer, obtainable via folder | ||
| :param n_mails: The number of mails to receive. Optional, default is 10 |
There was a problem hiding this comment.
Suggested change
| :param n_mails: The number of mails to receive. Optional, default is 10 | |
| :param n_mails: The number of mails to receive. Optional, default is 30 |
|
Mh, our CI servers might have been blacklisted by the OSM API. This happened to me in another project with a different API 🙈 |
|
Hmm we could test for something else? |
Signed-off-by: Jana Peper <jana.peper@nextcloud.com>
|
@janepie Could you rebase? :) |
Sure, but this is a little bit on hold right now as it's low priority and I wanted to have a look if I can polish it up a bit more. I could set it to Draft? |
|
Alright, sure :) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
based on nextcloud/mail#11425
adds