-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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: remove quotes from IMAP Folder Name #23242
Merged
Merged
Conversation
This file contains 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
Quotes were added around folder name as spaces would cause issue with IMAP. This extra quotes are also passed to query which sets the uidnext and uidvalidity. where condition is never met This values are used on subsequent pulls and without this values, it is considered first pull and all UNSEEN mails are synced. So even If user selected All in sync option it will always work as UNSEEN option. When UNSEEN is used it will mark all emails as seen on IMAP server when pulled.
maharshivpatel
requested review from
a team,
shariquerik and
ankush
and removed request for
a team and
shariquerik
November 16, 2023 10:15
github-actions
bot
added
the
add-test-cases
Add test case to validate fix or enhancement
label
Nov 16, 2023
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## develop #23242 +/- ##
===========================================
- Coverage 62.30% 62.29% -0.01%
===========================================
Files 770 770
Lines 73740 73754 +14
Branches 6349 6349
===========================================
+ Hits 45943 45945 +2
- Misses 24183 24195 +12
Partials 3614 3614
Flags with carried forward coverage won't be shown. Click here to find out more. |
maharshivpatel
added
backport version-14-hotfix
backport to version 14
backport version-15-hotfix
Backport the PR to v15
labels
Nov 21, 2023
mergify bot
pushed a commit
that referenced
this pull request
Nov 21, 2023
Quotes were added around folder name as spaces would cause issue with IMAP. This extra quotes are also passed to query which sets the uidnext and uidvalidity. where condition is never met This values are used on subsequent pulls and without this values, it is considered first pull and all UNSEEN mails are synced. So even If user selected All in sync option it will always work as UNSEEN option. When UNSEEN is used it will mark all emails as seen on IMAP server when pulled. (cherry picked from commit 0b4f06c)
mergify bot
pushed a commit
that referenced
this pull request
Nov 21, 2023
Quotes were added around folder name as spaces would cause issue with IMAP. This extra quotes are also passed to query which sets the uidnext and uidvalidity. where condition is never met This values are used on subsequent pulls and without this values, it is considered first pull and all UNSEEN mails are synced. So even If user selected All in sync option it will always work as UNSEEN option. When UNSEEN is used it will mark all emails as seen on IMAP server when pulled. (cherry picked from commit 0b4f06c)
This was referenced Nov 21, 2023
maharshivpatel
added a commit
that referenced
this pull request
Nov 21, 2023
Quotes were added around folder name as spaces would cause issue with IMAP. This extra quotes are also passed to query which sets the uidnext and uidvalidity. where condition is never met This values are used on subsequent pulls and without this values, it is considered first pull and all UNSEEN mails are synced. So even If user selected All in sync option it will always work as UNSEEN option. When UNSEEN is used it will mark all emails as seen on IMAP server when pulled. (cherry picked from commit 0b4f06c) Co-authored-by: Maharshi Patel <39730881+maharshivpatel@users.noreply.github.com>
maharshivpatel
added a commit
that referenced
this pull request
Nov 21, 2023
Quotes were added around folder name as spaces would cause issue with IMAP. This extra quotes are also passed to query which sets the uidnext and uidvalidity. where condition is never met This values are used on subsequent pulls and without this values, it is considered first pull and all UNSEEN mails are synced. So even If user selected All in sync option it will always work as UNSEEN option. When UNSEEN is used it will mark all emails as seen on IMAP server when pulled. (cherry picked from commit 0b4f06c) Co-authored-by: Maharshi Patel <39730881+maharshivpatel@users.noreply.github.com>
frappe-pr-bot
pushed a commit
that referenced
this pull request
Nov 22, 2023
# [15.2.0](v15.1.0...v15.2.0) (2023-11-22) ### Bug Fixes * added all options to add/remove section/column in section header ([05c7d67](05c7d67)) * added Dropdown component ([1349b74](1349b74)) * always show section header ([6ba6a99](6ba6a99)) * call auth hooks before raising error (backport [#23255](#23255)) ([#23268](#23268)) ([128fac7](128fac7)) * copy any flags if required in rename_doc() only on validate ([d790ec2](d790ec2)) * correct max file size in boot ([9e2b701](9e2b701)) * disable only save button, not the entire form ([349e4d7](349e4d7)) * don't copy value of "Is Standard" ([205ec4e](205ec4e)) * dont rename link fields in Virtual doctypes ([#23294](#23294)) ([99a3b4d](99a3b4d)) * email linking (backport [#23224](#23224)) ([#23226](#23226)) ([a02adef](a02adef)) * **email:** Allow users to pull all (read & unread) emails during initial sync ([a48fd81](a48fd81)) * **event/sync_communication:** add `distinct` while querying communication links ([1eb0f7e](1eb0f7e)) * Font for arabic and not supported languages ([#23234](#23234)) ([#23326](#23326)) ([f99b9d4](f99b9d4)) * Form Builder / Workflow Builder fixes (backport [#23238](#23238)) ([#23244](#23244)) ([c0ca3a8](c0ca3a8)) * honour max file size in upload file ([e63de38](e63de38)) * Ignore `None` returned by RQ job ([aeab15b](aeab15b)) * ignore case of filetype ([5bc6d1f](5bc6d1f)) * ignore invalid token so auth hooks can apply ([23e1a48](23e1a48)) * incorrect date range when relinking files ([#23197](#23197)) ([ab78bef](ab78bef)) * increase length for address lines ([2e126b6](2e126b6)) * **minor:** mobile menu & sidebar attachment text ([#23013](#23013)) ([#23014](#23014)) ([cd14e2a](cd14e2a)) * **minor:** web and timeline styles ([#23083](#23083)) ([c752ae5](c752ae5)) * move field description to doctype definition ([68f3748](68f3748)) * not able to edit private workspace (backport [#23241](#23241)) ([#23245](#23245)) ([f8eb101](f8eb101)) * remove link - modifying while iterating ([d5105ed](d5105ed)) * remove quotes from IMAP Folder Name ([#23242](#23242)) ([#23325](#23325)) ([9089f29](9089f29)) * removed unsed code from Column.vue ([406e0a5](406e0a5)) * review points when added more then once. ([#23239](#23239)) ([#23344](#23344)) ([11eb9a8](11eb9a8)) * send correct username in userinfo ([#23177](#23177)) ([#23178](#23178)) ([8c61435](8c61435)) * test docstatus change from workflow ([6e7aa5e](6e7aa5e)) * Update email_message in the send_welcome_email function to use the use_html check for Email Template ([78559be](78559be)) * web_form_list.js invalid condition_json ([2804192](2804192)) * **web_form:** check properties for title field as well ([64eda83](64eda83)) * **WebForm:** Add missing keyboard.js dep for some controls ([2fd07ee](2fd07ee)) * **WebForm:** Replace `get_cached_value` with `get_value` ([8989289](8989289)) * **web:** Send first_day_of_the_week and number_format in bootinfo ([a6308c3](a6308c3)) * **workflow:** avoid implicit docstatus altering ([c3a1912](c3a1912)) * workspace loading issue ([#23179](#23179)) ([e06f128](e06f128)) ### Features * Enabling Redirection to a Custom URL on Notification Click (backport [#22956](#22956)) ([#23318](#23318)) ([b52fac5](b52fac5)) * workflow state wise status override (backport [#23207](#23207)) ([#23315](#23315)) ([a8fab7b](a8fab7b)) ### Reverts * code control loosing focus when edit (for now reverting need better fix) ([8dfe6a7](8dfe6a7))
frappe-pr-bot
pushed a commit
that referenced
this pull request
Nov 22, 2023
# [14.56.0](v14.55.1...v14.56.0) (2023-11-22) ### Bug Fixes * added get_quarter_ending function in safe_exec ([2e46fc2](2e46fc2)) * Added option for setting default in Report filters ([19762d1](19762d1)) * call auth hooks before raising error (backport [#23255](#23255)) ([#23267](#23267)) ([8082865](8082865)) * correct max file size in boot ([c1a7008](c1a7008)) * disable only save button, not the entire form ([a7c7890](a7c7890)) * don't copy value of "Is Standard" ([db1399b](db1399b)) * dont rename link fields in Virtual doctypes ([#23293](#23293)) ([576b838](576b838)) * **email:** Allow users to pull all (read & unread) emails during initial sync ([64e10be](64e10be)) * **event/sync_communication:** add `distinct` while querying communication links ([f1e566e](f1e566e)) * honour max file size in upload file ([d0d5000](d0d5000)) * increase length for address lines ([6009545](6009545)) * move field description to doctype definition ([f06a823](f06a823)) * remove quotes from IMAP Folder Name ([#23242](#23242)) ([#23324](#23324)) ([2986702](2986702)) * remove unnecessary statuses from email queue and only append emails to sent if imap is enabled ([20c8ebe](20c8ebe)) * Resolve conflicts ([d0299af](d0299af)) * review points when added more then once. ([#23239](#23239)) ([#23343](#23343)) ([51dc38f](51dc38f)) * Translate user facing string ([fb9c3a3](fb9c3a3)) * web_form_list.js invalid condition_json (backport [#23277](#23277)) ([#23278](#23278)) ([c759880](c759880)) * **web_form:** check properties for title field as well ([7af17d5](7af17d5)) * **WebForm:** Replace `get_cached_value` with `get_value` ([ffab13c](ffab13c)) ### Features * workflow state wise status override (backport [#23207](#23207)) ([#23314](#23314)) ([0c77eac](0c77eac))
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Labels
add-test-cases
Add test case to validate fix or enhancement
backport version-14-hotfix
backport to version 14
backport version-15-hotfix
Backport the PR to v15
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.
Quotes were added around folder name as spaces would cause issue with IMAP.
These extra quotes are also passed to query which sets the uidnext and uidvalidity. where condition is never met
These values are used on subsequent pulls and without these values, it is considered first pull and all UNSEEN mails are synced.
So even If user selected All in sync option it will still always work as the UNSEEN option.
When UNSEEN is used it will mark all emails as seen on IMAP server when pulled.