Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Email] Changes to email reply API and other minor fixes #2409

Closed
wants to merge 2 commits into from
Closed

[Email] Changes to email reply API and other minor fixes #2409

wants to merge 2 commits into from

Conversation

darjoo
Copy link
Contributor

@darjoo darjoo commented Nov 27, 2024 •
edited by github-actions bot
Loading

Summary

  • Remove external id from reply functions. The external id should be added in the EmailMessage when creating.
  • Fix tests
  • Update parameter name Filter to UseFilter
  • Add captions to Email Retrieval Filters

Work Item(s)

Fixes AB#550315

@darjoo darjoo requested a review from a team as a code owner November 27, 2024 07:34
@github-actions github-actions bot added this to the Version 26.0 milestone Nov 27, 2024
@darjoo darjoo changed the title Remove external id from reply functions and fix some comments [Email] Changes to email reply API and other minor fixes Nov 27, 2024
@darjoo
Copy link
Contributor Author

darjoo commented Nov 28, 2024

Changes are in #2415.

Closing PR

@darjoo darjoo closed this Nov 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
AL: System Application
Projects
None yet
Milestone
Version 26.0
Development

Successfully merging this pull request may close these issues.
1 participant
@darjoo