[17.0][MIG] mail_tracking: Migration to 17.0 - Githubissues

OCA / social

Addons concerning odoo's social ERP features and messaging in general

https://odoo-community.org/project/99

GNU Affero General Public License v3.0

159 stars 603 forks source link

[17.0][MIG] mail_tracking: Migration to 17.0 #1367

Closed chienandalu closed 2 months ago

chienandalu commented 4 months ago

Ongoing tasks:

[x] Refactor webclient
[x] Fix server logic
TODO: Add more reactivity

This migration is ready for review. :tada:

You could still find tiny issues regarding the reactivity of the components, mainly when we set a failed message as reviewed in the search failed panel or in Discuss. We'll work on that.
A deep refactor of the webclient code has been made in core and thus a complete refactor of the webclient side for mail_tracking was needed.
We've been splitting the funcionality in component that can be esasily used across the core components, so hopefully everything is more modular now.
We're sitting on top of some new core features as well. Like a message search component to filter the failed messages.
Also regular server logic stuff was necessary, although it wasn't needed such a deep refactor as in the client side. Note that the tests are almost untouched in this part.

cc @Tecnativa TT49060

pedrobaeza commented 4 months ago

/ocabot migration mail_tracking

pedrobaeza commented 4 months ago

Please squash a bit administrative commits.

Bart-dh commented 3 months ago

Hi @chienandalu is this safe to use for production environments already?

chienandalu commented 3 months ago

Hi @chienandalu is this safe to use for production environments already?

It should be ready by now.

pedrobaeza commented 2 months ago

/ocabot merge nobump

OCA-git-bot commented 2 months ago

This PR looks fantastic, let's merge it! Prepared branch 17.0-ocabot-merge-pr-1367-by-pedrobaeza-bump-nobump, awaiting test results.

OCA-git-bot commented 2 months ago

Congratulations, your PR was merged at 2d458df82a4761d76fa907068d606409f8284236. Thanks a lot for contributing to OCA. ❤️