Skip to content

[271] Merge new emails into existing case#853

Merged
d-a-v-e merged 17 commits into
developfrom
feature/271-merge-new-email-to-existing
Feb 11, 2022
Merged

[271] Merge new emails into existing case#853
d-a-v-e merged 17 commits into
developfrom
feature/271-merge-new-email-to-existing

Conversation

@d-a-v-e

@d-a-v-e d-a-v-e commented Feb 8, 2022 •
edited
Loading

Copy link
Copy Markdown
Contributor

Changes in this PR

  • New journey for merging a new (auto-created via email) case's emails into an existing case
  • Does not allow a case to be merged if it is not in state :new
  • Does not allow a case to merged into itself
  • Adds Support::Case search via pg_search gem

Resources

Screencapture of UI changes

Screen.Recording.2022-02-11.at.10.25.11.mov

@viezly

viezly Bot commented Feb 8, 2022 •
edited
Loading

Copy link
Copy Markdown

This pull request is split into 4 parts for easier review.
👀 Review pull request on Viezly

Changed files are located in these folders:

  • /
  • app
  • config
  • lib

@d-a-v-e

d-a-v-e commented Feb 8, 2022

Copy link
Copy Markdown
Contributor Author

#383 Ability to merge a NEW email case into an existing case

@d-a-v-e d-a-v-e marked this pull request as ready for review February 11, 2022 08:36
@d-a-v-e d-a-v-e added the enhancement New feature or request label Feb 11, 2022
d-a-v-e
d-a-v-e commented Feb 11, 2022
View reviewed changes
Comment thread app/controllers/support/cases_controller.rb Outdated
@d-a-v-e d-a-v-e requested review from shafiqahmedkhan and removed request for kennyevil February 11, 2022 10:48
@d-a-v-e d-a-v-e enabled auto-merge (squash) February 11, 2022 10:49
d-a-v-e
d-a-v-e commented Feb 11, 2022
View reviewed changes
Comment thread CHANGELOG.md
@d-a-v-e d-a-v-e merged commit 7ac9cc5 into develop Feb 11, 2022
@d-a-v-e d-a-v-e deleted the feature/271-merge-new-email-to-existing branch February 11, 2022 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ryantk ryantk Awaiting requested review from ryantk
@peterdavidhamilton peterdavidhamilton Awaiting requested review from peterdavidhamilton
@shafiqahmedkhan shafiqahmedkhan Awaiting requested review from shafiqahmedkhan
1 more reviewer
@kennyevil kennyevil kennyevil approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@d-a-v-e @kennyevil