Skip to content

Conversation

@AmanDevelops
Copy link
Contributor

@AmanDevelops AmanDevelops commented Feb 1, 2025

Issue Referenced

closes #45

Describe the Changes

Change 1: src/assistant.py

  • Implemented a query processor to extract the recipient's email address, subject, and body from the user query.

Change 2: src/commands.py

  • Implementation of send email functionality

Type of Change

  • New feature (non-breaking change which adds functionality)

Additional Setup Requirements

  • user needs to provide additional details to src/email_config.json file and set SMTP password in environmental variable "DESKTOP_ASSISTANT_SMTP_PWD"

Necessary Conditions

  • I have commented on my code, particularly in hard-to-understand areas.
  • My changes generate no new warnings.
  • The changes do not involve any Private data leak/reveal.
  • Secrets, if any, are kept hidden via an environment file, whose structure has been explained via documentation.

Please make sure to check the following milestones before submitting the bug report!

@AmanDevelops
Copy link
Contributor Author

Untitled

@AmanDevelops
Copy link
Contributor Author

Annotation 2025-02-02 013906

@vihar-s1
Copy link
Owner

vihar-s1 commented Feb 5, 2025

@AmanDevelops can you specify how did you setup the server for testing the email feature ?

@vihar-s1 vihar-s1 merged commit d68353a into vihar-s1:main Feb 5, 2025
5 checks passed
@AmanDevelops AmanDevelops deleted the email branch February 5, 2025 18:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEAT]: Sending Emails

2 participants