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.

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

fix: strip comma, space from recipients before sending email for auto repeat #20940

Merged
merged 1 commit into from
May 9, 2023

Conversation

phot0n
Copy link
Contributor

@phot0n phot0n commented May 9, 2023

QueueBuilder by default splits the emails by comma, whitespace

@github-actions github-actions bot added the add-test-cases Add test case to validate fix or enhancement label May 9, 2023
@phot0n phot0n force-pushed the clean-recipients-auto-repeat branch from b7f106a to 16d6351 Compare May 9, 2023 12:52
@phot0n phot0n marked this pull request as ready for review May 9, 2023 12:54
@phot0n phot0n requested review from a team and shariquerik and removed request for a team May 9, 2023 12:54
@phot0n phot0n added backport version-14-hotfix backport to version 14 and removed add-test-cases Add test case to validate fix or enhancement labels May 9, 2023
@phot0n phot0n merged commit ab6e74e into frappe:develop May 9, 2023
14 of 15 checks passed
mergify bot pushed a commit that referenced this pull request May 9, 2023
@phot0n phot0n deleted the clean-recipients-auto-repeat branch May 9, 2023 13:01
phot0n added a commit that referenced this pull request May 9, 2023
… repeat (#20940) (#20945)

(cherry picked from commit ab6e74e)

Co-authored-by: Ritwik Puri <ritwikpuri5678@gmail.com>
frappe-pr-bot pushed a commit that referenced this pull request May 9, 2023
# [14.36.0](v14.35.0...v14.36.0) (2023-05-09)

### Bug Fixes

* ensure that `get_last_email` returns the most recent email (backport [#20711](#20711)) ([624f96b](624f96b))
* escape html from listview row title ([56bec1d](56bec1d))
* escape html from workspace title ([e68fc43](e68fc43))
* ignore virtual doctypes during data export ([#20891](#20891)) ([#20899](#20899)) ([d6bfaae](d6bfaae))
* make operator in link filters translatable (backport [#20884](#20884)) ([#20911](#20911)) ([1ec3bad](1ec3bad))
* message.py executing script ([#20887](#20887)) ([#20897](#20897)) ([1bcf5d4](1bcf5d4))
* **multi-pdf:** change response type to pdf ([997559c](997559c))
* **oauth:** add exp to idToken ([#20694](#20694)) ([#20903](#20903)) ([1a8e671](1a8e671))
* reload communication before re-save ([#20914](#20914)) ([#20921](#20921)) ([37a8ec0](37a8ec0))
* set default letterhead and print format ([a5a6965](a5a6965))
* strip comma, space from recipients before sending email for auto repeat ([#20940](#20940)) ([#20945](#20945)) ([042a1d2](042a1d2))
* translate lowercase operators to german ([#20912](#20912)) ([#20916](#20916)) ([c47b146](c47b146))
* type hints for get_address_display ([#20923](#20923)) ([#20924](#20924)) ([15df963](15df963))
* use smaller font only if the report doesnt have a standard print format ([#20878](#20878)) ([#20947](#20947)) ([35165d0](35165d0))

### Features

* helper method for address display ([#20900](#20900)) ([#20901](#20901)) ([f914770](f914770))
* telemetry using posthog (backport [#20825](#20825)) ([#20934](#20934)) ([bbe29ee](bbe29ee))

### Performance Improvements

* get all file data at once when downloading private file ([#20902](#20902)) ([e106594](e106594))
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 24, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport version-14-hotfix backport to version 14
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant