summaryrefslogtreecommitdiff
path: root/addons/mail/static/src/js
AgeCommit message (Collapse)Author
2020-10-14[FIX] crm,mail: clear breadcrumb when redirecting to activities from systrayKamesh Patel
Before this commit: When accessing activities from the systray, any existing breadcrumb-item should be clearer. After this commit: Clear breadcrumb-item when access activities from the systray. Task-2342246 closes odoo/odoo#59997 Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-10-07[FIX] mail: clean up relations between chatter and threadSébastien Theys
This will prevent various flicker when using previous/next. task-2243518 closes odoo/odoo#59375 X-original-commit: 1559e82e6b781601286003c2aff364dbb19c9bfc Related: odoo/enterprise#13868 Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-10-07[FIX]*: clear breadcrumb when redirecting to activities from systrayKamesh Patel
* mail,calendar,note Before this commit: When accessing activities from the systray, any existing breadcrumb-item should be clearer. After this commit: Clear breadcrumb-item when access activities from the systray. Task-2342246 closes odoo/odoo#59389 X-original-commit: 6dd181c789d22f99073fcd0a9b0fa2edccdfe0fc Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-09-29[FIX] mail: fix tour to allow public or private channel selectionSébastien Theys
task-2313321 closes odoo/odoo#58778 X-original-commit: f6a561a7e21772c6a25785c8b2428954d4bd152e Signed-off-by: Sébastien Theys (seb) <seb@odoo.com> Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-09-16[FIX] mail: do not fetch attachment style in QUnit testsAlexandre Kühn
Task-2339931 closes odoo/odoo#57869 X-original-commit: 7586f7756859bb0711eee8a82bf728a039ad0d72 Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-08-28[FIX] mail, *: fix opens and redirects, adapt messaging menu closeSébastien Theys
task-2244279 task-2262862 task-2274993 task-2277537 task-2282238 closes odoo/odoo#56745 X-original-commit: 109f202 Related: odoo/enterprise#12761 Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com> Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
2020-08-27[FIX] mail: 'Cannot read messagingMenu of undefined' tracebackAlexandre Kühn
Before this commit, on page reload, sometimes web client crashed with `Cannot read 'messagingMenu' of undefined`. This happens due to messaging components making use of `messaging` in `env` as if it was always set. In some rare cases, this is false, hence the crash. This happens on root components of messaging such as the messaging menu. Creation of `env.messaging` has to be async, in other to ensure all JS module that patches messaging are applied. The mounting of messaging root components depends on other parts of the web client like the SystrayMenu. So the proper fix is to guard `env.messaging` on these root components. Task-2238245 closes odoo/odoo#56682 X-original-commit: d1483ff3c83bb3146e80e5d65a27915c5b2a2bf9 Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
2020-08-19[FIX] mail: use right chatter field for chatter prop valueXavier Dubuc
task-2319706 closes odoo/odoo#56108 X-original-commit: 8955626b0f08be7364e715f0fdb91ef72a31f757 Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-08-14[IMP]hr_recruitment: Back2Basic - Improve Recruitment usabilityFrancois (fge)
Added a new tour for the module hr_recruitment Hr.job: Form view: - Using a notebook view for increased clarity Kanban view: - Added a toggle button to publish/unpublish a job offer. - In the menu, added the « Create Interview Form » button to create an interview form. Hr.applicant: Generale: - Use of the "Applicant: Acknowledgement" template by default when an application is moved to the "First Interview" stage. Form view: - Possibility to consult an interview after this one and not before - The "CREATE EMPLOYEE" button no longer directly creates an employee. The employee will be created if the "SAVE" button is pressed. Hr.employee: Generale: - Added of two new certifications. And some other minor style and name changes. closes odoo/odoo#55928 Related: odoo/enterprise#12420 Signed-off-by: Yannick Tivisse (yti) <yti@odoo.com>
2020-08-11[REF] mail: split emoji files to ease inheritanceThibault Delavallée
Purpose is to clearly distinguish * common: holding common code for emojis-enabled fields; * text emoji field: support of emojis in text fields; * char emoji field: support of emojis in char fields; Task ID-2224393 PR #49096
2020-08-11[IMP] mail : improve emoji widget code to ease overridesxO-Tx
This commit attempts to improve emoji widget css and code to make it easily overridable by other modules. This is done in preparation for extension by the SmsWidget that will enable the emojis support. To improve the emoji widget, the following changes have been made * remove the spacing after emoji fields by setting widget dropdown height to 0 (emoji dropdown has relative position + (height > 0) which leaves an empty space after widget fields); * hide Emoji widget when field has "invisible modifier"; * move the code that adds the emoji dropdown to another method, allowing to better understand code organization; Task ID-2224393 PR #49096
2020-08-05[FIX] mail: clean up JS relations by storing recordsSébastien Theys
Remove the support and the conversion from record to localId and back from localId to record. Part of task-2308312 closes odoo/odoo#55481 X-original-commit: fc5e8be4e65518770a281f0dffae07854d20073a Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-07-31[FIX] mail: improve performance of JS write, especially for x2mSébastien Theys
- store `x2many` values as `Set`. This is conceptually more correct because duplicates were already not allowed, and using a `Set` allows for much faster operations such as checking the presence of a record in the relation, which is done at every write, for every linked record. - `stringify` in `write` was slow and unnecessary because we can always assume that new non-primitive should be provided to `write` - remove field values from store, in favor of just storing a counter, which is enough to trigger renders. This allows to get rid of one Proxy level around data, which was slower, but which also prevented from using `Set`. - write in store exactly once per update cycle, to trigger one `useStore` call on each component. Records keep `__state` but as a local counter, to let `useStore` know which records have been updated (easier to compare a counter than comparing each and every field value). Results ======= The worse case test (using a lot of x2m) that was used to benchmark this PR `form_renderer_tests.js` of `mail_enterprise` went from taking around 4s before this commit, to around 2.5s after this commit. And the worse `_setRelationLinkMany2Many` previously took around 250ms, and it now takes around 50ms. The whole `mail` QUnit suite took around 55s before, now around 42s. Part of task-2308312 closes odoo/odoo#55248 X-original-commit: 4db0a10d85534ef896f6bcbf13bca4983a6e2003 Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-07-29[FIX] mail: fix messaging locale update on translation bundle changeSébastien Theys
closes odoo/odoo#55167 X-original-commit: c3eb3f04e06cea84dd98323b6d10867dead5b953 Signed-off-by: Sébastien Theys (seb) <seb@odoo.com> Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-07-22[FIX] mail: update and re-render the activity viewHiral Bhavsar
Before this commit: when props change, the internal state of the widget not updated for the activity view. So it creating many issues: - The count of the progress bar doesn't go to update while scheduling the activity of the same type. - Date displayed on cell not getting updated while changing the due date by clicking on the cell. This commit will update legacy widget from owl component, one need to override renderWidget and updateWidget instead of render and update method as this has been changed from commit https://github.com/odoo/odoo/commit/f2de55322e4d417815e6757517163dd0cff544cc After this commit: the acivity view will be updated based on activity changes Task-2238551 PR https://github.com/odoo/odoo/pull/54734 X-original-commit: 02c7d121935ab1b26d896e31eb0c90bf2b7abff8
2020-07-13[IMP] web,*: do not deploy services in testsAaron Bohy
*mail,point_of_sale,web_editor,website In the tests, required services are deployed for each test independently. There is no need to have, in additon, all services deployed globally. Worse, it could conflict and lead to unexpected results. This commit ensures services are no longer deployed globally in tests. It turns the module 'web.env' into a declarative module with no side-effect, by moving the service deployment to main.js, which isn't added to the tests page. Task 2287397 closes odoo/odoo#53817 Signed-off-by: Lucas Perais (lpe) <lpe@odoo.com>
2020-06-29[IMP] hr_timesheet: Improve the usability of timesheet portalqho
1. Generic timesheets improvements - Portal /my/timesheets - Add a group bys task, date, employee - Add search in project, name, employee, task * The task column should not be visible if none of the timesheets have the Task field set - If a group by task, date, employee or project is applied, the corresponding column should not be visible 2. Timesheets from sub-tasks on parent tasks - Portal /my/tasks - Display the 'Total Hours' spent on a task at the end of the list of timesheets - Below the list of timesheets for the task, display another table detailing the respective timesheets (and total hours) of each sub-task linked to this task (= 1 table/list per sub-task) * Title should be: 'Timesheets for sub-task: xyz' * only visible if there are sub-tasks linked to this task * only visible if total hours > 0 3. Timesheets on Invoices - Portal /my/invoices - Add Project + task fields * Task field only visible if set on one of the timesheet entries * Project field only visible if there are at least 2 different projects in these timesheet entries 4. Correct visibility for the portal users - Portal users is allowed to see timesheets if the users are added in allowed portal users list in the project - When a portal user is added as the message follower of the project, the user is added to allowed portal users as well - When a portal user is removed from allowed portal users, the user is also removed from message followers closes odoo/odoo#47248 Taskid: 2201908 Signed-off-by: Yannick Tivisse (yti) <yti@odoo.com>
2020-06-26[FIX] mail: replace <br/> by space in thread previewXavier Dubuc
task-2284905 closes odoo/odoo#53718 X-original-commit: 80504fabd43832c5495ded3afc4b364c71d3df70 Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-06-22[REF] *: adapt code to web changesAaron Bohy
Some changes in AbstractModel have been done to ease the implementation of the sample data feature. This commit reverberates those changes in other addons. Part of task 2232801 X-original-commit: dd8069a3c80570e7fa03a80dbb100ab0a182692a
2020-06-15[REF] mail, *: refactor messaging with OWLSébastien Theys
This commit is a significant rewriting of client-side discuss, chatter, chat window, and messaging menu using OWL. The behavior should be broadly the same, with some slight functional changes here and there. From a technical standpoint, the code of messaging is mainly organized in 2 main groups of modules: - models, which are logical entities that depict the client-side state of messaging as a whole. - components, which are in charge of displaying information from models. This refactoring also introduces new JS guidelines regarding folder structure (/static) and naming rules for JS modules. Community PR: https://github.com/odoo/odoo/pull/39023 Enterprise PR: https://github.com/odoo/enterprise/pull/6249 Task-1914207 This PR is a collaborative work by Alexandre, Julien, Sébastien and Xavier, with the precious help of Lucas to speed it up towards the end. closes odoo/odoo#39023 Related: odoo/enterprise#6249 Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com> Co-authored-by: Alexandre Kühn <aku@odoo.com> Co-authored-by: Julien Giannone <jgi@odoo.com> Co-authored-by: Lucas Perais <lpe@odoo.com> Co-authored-by: Sébastien Theys <seb@odoo.com> Co-authored-by: Xavier Dubuc <xdu@odoo.com>
2020-06-15[IMP] mail, sms: support keydown based change event in char fields using an ↵Aymane (ayt)
option PURPOSE Avoid flickering and incorrect display of warning box in SMS when updating the number to contact. SPECIFICATIONS Currently when an user uses composer to send an SMS, number is validated at onchange. It leads to two issues * jump of content as a warning box appears; * visual warning displayed after user may have clicked on "Send" button, leading to warning not being useful; We choose to validate number in real time in order to give directly a feedback to the user. It also lessens flickering on composer screen. This is done by allowing to trigger the 'change' event on key down. This requires a specific onchange_on_keydown option to be used on char field. LINKS Task ID 2229050 PR #51159 closes odoo/odoo#52962 Signed-off-by: Thibault Delavallee (tde) <tde@openerp.com>
2020-06-12[IMP] web,*: properly define env in the frontendAaron Bohy
*bus,mail This commit extracts the common basis of the env to use both in the frontend and the backend. The public env now contains most of the features that were previously (only) in the webclient env. Moreover, the PublicRoot no longer uses the ServiceProviderMixin, such that services are only deployed once, in the (public) env. Owl components can now be defined in the website, and rely on a properly built env. Legacy widgets still works as they access services through the PublicRoot (via trigger_up) which redirects those requests to the env.
2020-06-12[IMP] *: update notifications to optional titleJulien Mougenot
closes odoo/odoo#52826 Related: odoo/enterprise#11084 Signed-off-by: Aaron Bohy (aab) <aab@odoo.com>
2020-06-09[REM] im_support: remove im_support moduleSébastien Theys
The goal of the module was to communicate with livechat operators on our production server, except the feature is disabled on our prod and we don't plan to reactivate it any time soon. closes odoo/odoo#52691 Related: odoo/upgrade#1313 Signed-off-by: Christophe Simonis <chs@odoo.com>
2020-06-11[FIX] mail: fix redirect to mention in ThreadWindowjvm-odoo
Issue - Install Contacts - In systray, open #general - Mention anyone (except your user) - Send - Click on the mention It does nothing Cause In thread_window _onRedirect, we pass the _open method bounded to this. But this = the #general thread window so it will open the general thread again. As it is already open, it does nothing Solution Modify the open method in order to receive the returned channelID and pass it to the call OPW-2265580 closes odoo/odoo#52834 X-original-commit: 5957aa317e8bb5b0041a1b5d8256cb3ff7d9fe42 Signed-off-by: Jason Van Malder (jvm) <jvm@odoo.com>
2020-06-10[IMP] mail: few improvements on activity widgetHiral Bhavsar
PURPOSE Display the activity type through the icon of the widget and change the place to display to whom the activity is assigned (if it is not assigned to the current user). SPECIFICATION Display the deadline with a prefix as responsible if the activity assigned to another user than the current one (uid). Add a new 'activity_type_icon' field to the mail mixin to get the 'icon' of the type (fa-phone, fa-users, etc...) and for the list_activity widget, display it as an icon instead of the default 'fa-clock-o' icon. LINKS PR https://github.com/odoo/odoo/pull/52252 Task-2267579 closes odoo/odoo#52252 Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-06-08[FIX] mail: fix mark several messages as readjvm-odoo
Issue - Install Discuss & any app with a chatter - Have 2 session, admin & demo - Clear admin systray messages As demo: - Discuss > #general - Ping (@...) admin 2 times As admin: - Go in the app you installed - Open systray messages - Mark messages as read It doesn't work Cause When marking as read we get the message-ids as a string and if you have several ids it will be ["1,2"] which is not supported by the back-end Solution The back-end should receive an array like [1, 2] OPW-2234671 closes odoo/odoo#52608 X-original-commit: fbe91697fb1bc2ade5eba034b97bd671eeaf200b Signed-off-by: Jason Van Malder (jvm) <jvm@odoo.com>
2020-05-25[FIX] mail: fix error when clicking on follower icon in chatterAnh Thao Pham (pta)
- Go to any page where there is a chatter - Click on the followers list icon to display all followers - Click on the icon (not the name) of a follower This generates an error. The redirect is executed in JS by retrieving the model and the id added to the <a> element. When clicking on the follower icon, we are not clicking on the <a> element but on a child. The model and the id have to be retrieved from its parent element. opw-2256795 closes odoo/odoo#51790 X-original-commit: 11c4a2907e635662216ebc0e745f0ce9d16771cb Signed-off-by: Nicolas Lempereur (nle) <nle@odoo.com>
2020-05-22[FIX] mail: Load record form view with user idLucas Lefèvre
fixes #45425 closes odoo/odoo#51717 X-original-commit: a210d68af04490c0daa535e3151ed5edbd431898 Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-05-22[FIX] mail: avoid scrolling "NavBar" when dropdown openRomeo Fragomeli
Before this commit, when a SysTray dropdown is open the user can scroll the element behind the dropdown. After this commit, we disable the scroll until we close the dropdown. Steps to reproduce: * Open Odoo on "Mobile" * Open the Activity SysTray * Scroll the "NavBar" menu (=> Bug) Note this fix is also applied in Desktop, as for me the "NavBar" shouldn't never scroll on Desktop. Task ID: 2231956 Task ID: 2234042 closes odoo/odoo#51716 X-original-commit: 7bb155991b9daec37edf73ef90cbad26268d76bb Signed-off-by: Adrien Dieudonné (adr) <adr@odoo.com> Signed-off-by: rfr-odoo <rfr-odoo@users.noreply.github.com>
2020-05-20[IMP] mail: display user avatar in chat-box headerHiral Bhavsar
PURPOSE We are currently working toward turning every user avatar (i.e. the round picture of the user) in a shortcut to open the chatbox to that user. It only makes sense to also have the user avatar in the header of the chatbox (like in Facebook) SPECIFICATION Replace the current IM status colored dot of the chatbox header by the same avatar/im_status compound currently used in the chatter. LINKS Closes https://github.com/odoo/odoo/pull/51258 Task-2256061 closes odoo/odoo#51258 Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-05-18[FIX] mail: do not eat all document.body.drop eventNicolas Lempereur
In fb051d3420 there was some changes on drag and drop of chatter windows, but this would break the drag and drop of summernote because it wait for document.drop event for hiding the drop zone. opw-2259522 closes #51448 closes odoo/odoo#51478 X-original-commit: 4aaf5b1f562f4cd30e05b020741eb936f6e20a20 Signed-off-by: Nicolas Lempereur (nle) <nle@odoo.com>
2020-05-07[FIX] mail: fix "open_attachments" option's attachments not loadedjvm-odoo
Issue - Install Recruitment - Go on any application - Add an attachment - Go back, refresh - Returns on the application The attachment box is open but the attachments are not loaded. You must toggle it again to load them. Cause When opening the box, this.attachments is empty so no attachments are shown. Solution Load the attachments if the option is present. It will not cause supplementary RPCs thanks to "_areAttachmentsLoaded" OPW-2246116 closes odoo/odoo#50825 X-original-commit: 40cea06301505f0f931b5cfce1da0f05760f8dcc Signed-off-by: Jason Van Malder (jvm) <jvm@odoo.com>
2020-05-05[IMP] mail: add ListActivity widgetAaron Bohy
This commit adds a tweaked version of the kanban_activity widget for the list view. This widget displays the summary of the next activity (and fallbacks on the activity type if there is no summary). It can be set by defining widget='list_activity' on field activity_ids. Part of task 2195254
2020-05-05[REF] mail: lint activity.jsAaron Bohy
2020-05-05[IMP] mail: add Many2OneAvatarUser field widgetAaron Bohy
This widget is an extension of Many2OneAvatar, designed for many2one fields pointing to the 'res.users' model, or a model having itself a many2one field pointing to 'res.partner'. With this widget, when the avatar is clicked, we open a DM chat window with the corresponding partner. If the user clicked on itself, we open a blank chat window for the sake of consistency. In kanban views, this widget only displays the avatar. Part of task 2195254
2020-05-05[REF] mail,hr: move openDMChatWindow to MailManagerAaron Bohy
It will be used by the Many2OneAvatarUser widget defined in the next commit.
2020-04-30[IMP] mail, crm: add view types for crm.lead model when coming from activity ↵Sébastien Mottet (oms)
menu Purpose of this commit is to allow customization of view types available when using the activity menu. By default only kanban, list and form are available as before. Calendar, pivot, graph and activity have been added for leads, allowing better user experience in their use of activities. Task ID: 2228451 Community PR odoo/odoo#50042 Enterprise PR odoo/enterprise#10145 Signed-off-by: Thibault Delavallee (tde) <tde@openerp.com>
2020-04-17[IMP] mail, sms, snailmail, test_mail_full: adapt mail.notificationSébastien Theys
The purpose is to have more common code for failure and notifications, with less override in `sms` and `snailmail`. task-2176017 closes odoo/odoo#44170 Related: odoo/enterprise#9140 Related: odoo/upgrade#918 Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-04-16[REF] web,mail: use patchMixin and patch instead of utils.patchMichael Mattiello (mcm)
This commit replaces the use of utils.patch by the new class patcher and makes the renderers that are converted to owl patchable. closes odoo/odoo#47700 Related: odoo/enterprise#9551 Signed-off-by: Géry Debongnie (ged) <ged@openerp.com>
2020-03-19[FIX] mail: correctly parse datetime returned from serverPierre Paridans
When a user is Out-of-Office, the return date is displayed as "Invalid date" on Safari (both iOS & macOS). The parsing of the date returned by the server was done using the Date constructor and passing the string as-is. This constructor delegates the parsing to the `Date.parse` method. But the DateTime format used to stringify dates in Odoo doesn't fully follow the ISO 8601, as required by `Date.parse()`: expected format: "YYYY-MM-DDTHH:mm:ss.sssZ" actual format: "YYYY-MM-DD HH:mm:ss" Also, Safari applies more strictly to this requirement than Chrome & Firefox, resulting in a parsing error in the former browser. This commit fixes it by using the utility function `str_to_datetime` from `web.time` to correctly parse this date, independently of the browser used. NB: as this issue is browser dependent, no test has been added. References: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Date/Date https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Date/parse https://tc39.es/ecma262/#sec-date-time-string-format Task ID: 2198486 closes odoo/odoo#47987 X-original-commit: b85a44aea9617b9018dd80d5f8159354bbc88f09 Signed-off-by: Adrien Dieudonné (adr) <adr@odoo.com>
2020-03-18[REF] mail: update discuss helpersJulien Mougenot
Update the 'createDisuss' helper to be appended to the DOM using 'dom.prepend' to benefit of its 'on_(attach|detach)_callback' calls. This allows the mail test suite environment to be closer to the actual action behaviour (having those methods called each time it is created/destroyed). Some tests in which these methods were manually called have been adapted. Also, one of the tests was destroying the action too quickly and exposed an issue within the Discuss action, in the 'on_detach_callback' method. It has thus been fixed as well.
2020-03-13[REF] *: Adapt all modules to new control panelJulien Mougenot
Adaptation of the code throughout all modules to properly instantiate the newly refactored control panel and update it accordingly. Part of task 2196029 Co-authored-by: Mathieu Duckerts-Antoine <dam@odoo.com>
2020-03-05[FIX] mail: do not escape feedback before sending to backendAndrea Grazioso (agr-odoo)
Open every model with the chatter Schedule an activity Click on "Done" button and insert a feedback with an accent/quote. Escaped sequence will appear as result. In v13 the html feedback field which was used to store feedback was replaced by the mail subsystem, so the value should be sent to the message_post_with_view unescaped opw-2209081 closes odoo/odoo#46985 X-original-commit: d66c676e6a49cbf4cfcdae014c3b5e80dd7920d2 Signed-off-by: Nicolas Martinelli (nim) <nim@odoo.com>
2020-03-10[FIX] mail: prevent hidden chat window close from opening home menuSébastien Theys
opw-2210274 closes odoo/odoo#47245 X-original-commit: ca695425a2b06ffba15d7063fb0b8ea994f751d6 Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-03-09[FIX] mail: allow to switch on form viewHiral Bhavsar
Currently, when clicking on the systray activity action icon and click on any record from the activity, kanban or list view, not redirecting to its form view. This commit adds the form view for that action and now click on record will redirect to its form view. task-2198480 Closes https://github.com/odoo/odoo/pull/46877 closes odoo/odoo#47208 X-original-commit: 3fd30316c0af9a6ca2ea4962f2e2f832e68dee54 Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-03-05[REV] mail: Move the attachment icon on the leftPierre Paridans
This revert of commit odoo/odoo@9cd3bcd4408be76168e1ab4c08f2d8e187096d79 is due to the attachment's button needing to be kept on the right side of the chatter's topbar. Task ID: 2193950 closes odoo/odoo#46958 X-original-commit: d458aca28c25879ca9878b0628f2a6ea703504ef Related: odoo/enterprise#9027 Signed-off-by: Pierre Paridans <pparidans@users.noreply.github.com>
2020-02-25[FIX] mail: improve discuss new channel handling in mobileAlexandre Kühn
Revision on https://github.com/odoo/odoo/commit/8e5b6ab02aa95c204ea7bd241e6b8944d5972d43 Intent of commit above was to remove server-side synchronization of chat windows in mobile. To do so, it prevented sending RPCs for chat window state in mobile, in addition to not automatically adapt state of chat windows in mobile from longpolling notification. This implementation, while functionally working as intended, had disturbing intermediate logic that could potentially lead to bugs in the future. Specifically, on mobile, new channel did not directly trigger a re-render of discuss widgets. However, and thankfully, the related new message was triggering the rendering, so that UI correctly has channel and its notification. This commit simply improves code so that new channel always triggers a rendering, even in mobile, like before the commit above. Of course, to preserve the intent of not synchronizing chat windows in mobile, new detached channels must still not auto-open a chat window, and this commit clarifies this intended behaviour in the code. X-original-commit: 1046d063ca46aacb1e186599b5d4a02ef9c8e18d
2020-02-20[FIX] mail: activity popup is outside the screenRomeo Fragomeli
Before this commit, the popup activity in kanban was rendered using the position of the button. So the left popup was nearly in the center of the screen but the width is too large to be contained on the screen. After this commit, in mobile, the popup activity in kanban will be placed on the left of the button to avoid to go outside of the viewport. Steps to reproduce: * Go to sale app (in mobile) * Click on the activity button of a kanban card (clock icon) BUG Task ID: 2198429 closes odoo/odoo#45860 X-original-commit: 392c8a57df798d92b8be73d743b690891cf2fda2 Signed-off-by: Pierre Paridans <pparidans@users.noreply.github.com>
2020-02-18[FIX] mail: no crash on receiving needaction on messaging not readyAlexandre Kühn
Before this commit, when receiving a needaction message while messaging was not ready (i.e. `/mail/init_messaging` not resolved), the web client crashed with the following error message: ``` TypeError: Cannot read property 'incrementMailboxCounter' of undefined ``` This happens due to reception of needaction message incrementing counter of mailbox Inbox, but this mailbox is only added after messaging is ready. This commit fixes the issue by directly adding inbox mailbox on starting the mail service. closes odoo/odoo#45442 X-original-commit: 741acb9f48f2d3542549791a8fd7d6fb318bc862 Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>