-
Notifications
You must be signed in to change notification settings - Fork 10.9k
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] Save edited tags for omnichannel departments #26481
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #26481 +/- ##
===========================================
+ Coverage 38.54% 38.70% +0.16%
===========================================
Files 758 758
Lines 18827 18829 +2
Branches 1453 1453
===========================================
+ Hits 7256 7288 +32
+ Misses 11355 11325 -30
Partials 216 216
Flags with carried forward coverage won't be shown. Click here to find out more. |
…edit-department-tag
…t.Chat into fix/edit-department-tag
apps/meteor/client/views/omnichannel/departments/EditDepartment.js
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think comparing the old x new value would be the best solution
btw dont forget e2e tests!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Function-wise seems to be working 👀
apps/meteor/client/views/omnichannel/departments/EditDepartment.js
Outdated
Show resolved
Hide resolved
…t.js Co-authored-by: Guilherme Gazzo <guilhermegazzo@gmail.com>
Co-authored-by: Aleksander Nicacio da Silva <6494543+aleksandernsilva@users.noreply.github.com> Co-authored-by: Guilherme Gazzo <5263975+ggazzo@users.noreply.github.com>
Proposed changes (including videos or screenshots)
A simple workaround for the main problem, when editing a department must be possible to save addition or remotion of tags.
Issue(s)
Steps to test or reproduce
Further comments