Skip to content

Conversation

@barkat-matthias
Copy link

Migrate queue_job_batch et test_queue_job_batch

etobella and others added 30 commits June 4, 2025 14:57
[UPD] Update queue_job_batch.pot
standard migration and some little ux improvements: now hide
button is a fa-check button that on click just marks as read
the batch, do not navigate to it and updates immediately
the systray.
* The batch enqueued method is removed, to better reflect the fact that the
  batch's jobs are executed immediately.

* The batch state `draft` is is renamed to `pending`, for the same purpose.

* The batch's `is_read` field is initialized as `False` (default value), and
  only set to `True` when it's actually read.

* We now use `identity_exact` for the batch `check_state` delayed method.

* The widget has been reimplemented due to the heavy refactoring made in the
  core `mail` app: odoo/odoo@7710c333
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: queue-18.0/queue-18.0-test_queue_job_batch
Translate-URL: https://translation.odoo-community.org/projects/queue-18-0/queue-18-0-test_queue_job_batch/
[UPD] Update queue_job_batch.pot
* incorrect sintax in security rule
* change missleading name of security rule
standard migration and some little ux improvements: now hide
button is a fa-check button that on click just marks as read
the batch, do not navigate to it and updates immediately
the systray.
* The batch enqueued method is removed, to better reflect the fact that the
  batch's jobs are executed immediately.

* The batch state `draft` is is renamed to `pending`, for the same purpose.

* The batch's `is_read` field is initialized as `False` (default value), and
  only set to `True` when it's actually read.

* We now use `identity_exact` for the batch `check_state` delayed method.

* The widget has been reimplemented due to the heavy refactoring made in the
  core `mail` app: odoo/odoo@7710c333
oca-ci and others added 4 commits June 4, 2025 14:57
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: queue-18.0/queue-18.0-queue_job_batch
Translate-URL: https://translation.odoo-community.org/projects/queue-18-0/queue-18-0-queue_job_batch/
Currently translated at 100.0% (62 of 62 strings)

Translation: queue-18.0/queue-18.0-queue_job_batch
Translate-URL: https://translation.odoo-community.org/projects/queue-18-0/queue-18-0-queue_job_batch/it/
@barkat-matthias barkat-matthias marked this pull request as draft June 9, 2025 14:41
@barkat-matthias barkat-matthias force-pushed the 17.0-mig-queue-job-batch branch from 9665bc5 to 411eeb3 Compare June 9, 2025 15:14
@barkat-matthias barkat-matthias changed the title 17.0 mig queue job batch [17.0] mig queue job batch Jun 25, 2025
@barkat-matthias barkat-matthias marked this pull request as ready for review June 25, 2025 07:20
Copy link
Contributor

@petrus-v petrus-v left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As far I understand v16.0 and version v18.0 has notifications system while queue_job.batch state changed. Is it desired to drop that support while migrate the module ?

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@github-actions
Copy link

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Oct 26, 2025
Copy link

@mostafabarmshory mostafabarmshory left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This addon accepted in both 18 and 16 but not in 17. Is there any issue? It could be merged with main branch.

@mostafabarmshory
Copy link

@guewen

Could you check this PR and merge with the main branch please?

@mostafabarmshory
Copy link

@barkat-matthias @petrus-v @StephaneMangin

Dear guys

this PR is ok and checked with other. Would you merge this with the branch 17.0, please?

@mostafabarmshory
Copy link

@sbidoul

this RP needs merge command. Could you merge the RP with branch 17.0, please?

@github-actions github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Nov 9, 2025
@ryanc-me
Copy link

@AnizR any chance we can merge this? It's been reviewed, CI is all green, so ready to go I think :)

@petrus-v
Copy link
Contributor

@barkat-matthias @petrus-v @StephaneMangin

Dear guys

this PR is ok and checked with other. Would you merge this with the branch 17.0, please?

We are not allowed to merge this PR. I think it's better to ping @OCA/queue-maintainers team !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.