Skip to content
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

[17.0][MIG] rma: Migration to 17.0 #385

Merged
merged 98 commits into from
May 15, 2024
Merged

[17.0][MIG] rma: Migration to 17.0 #385

merged 98 commits into from
May 15, 2024

Conversation

peluko00
Copy link

@peluko00 peluko00 commented May 3, 2024

Module migrated to version 17.0

cc https://github.com/APSL 154788

@miquelalzanillas @lbarry-apsl @javierobcn @mpascuall please review

Ernesto Tejeda and others added 30 commits May 3, 2024 13:08
[UPD] Update rma.pot

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: rma-12.0/rma-12.0-rma
Translate-URL: https://translation.odoo-community.org/projects/rma-12-0/rma-12-0-rma/
…flow + Translated using Weblate (Spanish)

[FIX] rma: views permissions

Regular users don't have permissions to rma models, so we should avoid
loading views that lead to permission errors.

TT24986

rma 12.0.1.1.0

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: rma-12.0/rma-12.0-rma
Translate-URL: https://translation.odoo-community.org/projects/rma-12-0/rma-12-0-rma/

[FIX] rma: portal views access errors

- Portal mail thread needs token config.
- Unpublished products will raise AccessError on RMAs portal views for
portal users due to record rules.
- Ensure active_id when getting actions in rma, since we could come from
a context that pollutes the expected active rma id.

[IMP] rma: teams flow

- If no RMA Team is set, we'll assign a default one to the new RMA.
- A sequence is now used to search for the top team and assign it.
- No default user is assigned when it's not in the context (i.e. portal
rmas).

[UPD] Update rma.pot

rma 12.0.1.2.0

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: rma-12.0/rma-12.0-rma
Translate-URL: https://translation.odoo-community.org/projects/rma-12-0/rma-12-0-rma/

Translated using Weblate (Spanish)

Currently translated at 96.2% (253 of 263 strings)

Translation: rma-12.0/rma-12.0-rma
Translate-URL: https://translation.odoo-community.org/projects/rma-12-0/rma-12-0-rma/es/
- Fix thrown error when trying to download a picking from the portal.
- Add the hook method to prepare RMA values ​​from the return pick wizard.
- Add the access rule for portal users.
- Show the portal 'Request RMAs' button on the sales page only to users
related to the sales order.

[UPD] Update rma.pot

rma 12.0.1.3.0

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: rma-12.0/rma-12.0-rma
Translate-URL: https://translation.odoo-community.org/projects/rma-12-0/rma-12-0-rma/
Currently translated at 100.0% (263 of 263 strings)

Translation: rma-12.0/rma-12.0-rma
Translate-URL: https://translation.odoo-community.org/projects/rma-12-0/rma-12-0-rma/es/
- Avoid permission blockings on users who doesn't have RMA permissions.
- Add activities view on RMA list.

[UPD] Update rma.pot

rma 12.0.1.3.1
Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: rma-12.0/rma-12.0-rma
Translate-URL: https://translation.odoo-community.org/projects/rma-12-0/rma-12-0-rma/
- Now it's possible to open several RMAs in a sale order from the portal
- A new comment button has been added to allow the portal user to enter
relevant information like serial numbers o issue description.
- If the requested operation isn't set no RMA will be opened
- The RMA product qty is now a numeric control with limits according to
the qty available to return

[FIX] rma,rma_sale: fix linter errors

[UPD] Update rma.pot

rma 12.0.1.4.0

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: rma-12.0/rma-12.0-rma
Translate-URL: https://translation.odoo-community.org/projects/rma-12-0/rma-12-0-rma/
Not indicating account.invoice, it takes the default that is the vendor bill view.
We force the suitable one.
…ype use existing lot by default.

[IMP] rma: set rma to received on invoice delete.

Remove 'waiting_refund' rma state.
RMAs go from received to refunded.
When the linked refund is deleted the rma is set to received.

[UPD] Update rma.pot

rma 12.0.1.4.2

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: rma-12.0/rma-12.0-rma
Translate-URL: https://translation.odoo-community.org/projects/rma-12-0/rma-12-0-rma/

[IMP] rma: incoming picking type use existing lot by default.

rma 12.0.1.4.3
…ngs:

[FIX+IMP] rma: usability

* IMP - Now the description will be an html son we can show rich styles
in the customers report.
* FIX - On locked sale orders it was need to unlock them to be able to open an RMA.
* IMP - Make the description label visible in the backend form so the
user can easily spot it.
* IMP - Added date and deadline filters.
* IMP - Added pending RMAs filter.
* IMP - Added late RMAs filter.
* IMP - Added danger decoration in tree view

rma 12.0.1.5.0

[UPD] Update rma.pot

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: rma-12.0/rma-12.0-rma
Translate-URL: https://translation.odoo-community.org/projects/rma-12-0/rma-12-0-rma/

Translated using Weblate (Spanish)

Currently translated at 100.0% (268 of 268 strings)

Translation: rma-12.0/rma-12.0-rma
Translate-URL: https://translation.odoo-community.org/projects/rma-12-0/rma-12-0-rma/es/

[FIX] rma: return permissions

When a user with no stock manager permissions was doing the return, an
access error would raise.

rma 12.0.1.5.1

[FIX] rma: multicompany returns

The move was created with the default company

[UPD] Update rma.pot

rma 12.0.1.5.2

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: rma-12.0/rma-12.0-rma
Translate-URL: https://translation.odoo-community.org/projects/rma-12-0/rma-12-0-rma/

[FIX] rma: non rma users confirm

The sudo had to be before calling the records

rma 12.0.1.5.3

[FIX] rma: pass company as record to procurement

This upstream commit expects `company_id` as a record instead of as an
integer id: odoo/odoo@836ff55

rma 12.0.1.5.4

[FIX] rma: internal users portal

Internal users with no RMA permissions got an access error when entering
the portal

[UPD] Update rma.pot

rma 12.0.1.6.0

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: rma-12.0/rma-12.0-rma
Translate-URL: https://translation.odoo-community.org/projects/rma-12-0/rma-12-0-rma/
Currently translated at 62.2% (168 of 270 strings)

Translation: rma-13.0/rma-13.0-rma
Translate-URL: https://translation.odoo-community.org/projects/rma-13-0/rma-13-0-rma/ro/
Currently translated at 1.8% (5 of 270 strings)

Translation: rma-13.0/rma-13.0-rma
Translate-URL: https://translation.odoo-community.org/projects/rma-13-0/rma-13-0-rma/zh_CN/
When a sale line has a phantom product (mrp kits) the RMA would not be
possible as the wizard couldn't pair the components moves with the
product in the line. With this approach, we can at least return the
spare components of the original kit line.

We also need some hooks to intervine in the main methods, like in
invoicing.
Allow to set the desired shipping address where the goods must be
returned after the RMA is processed.
Now we can configure if an automatic notification should be sent when we
receive the goods from an RMA in our warehouse

If we've got `rma_sale` or `website_rma` we can also configure draft
notifications so when the customer places an RMA from the portal the
receive an acknowledge email.

TT29595
Now we can decide which tags are visible for the customer. This way, we
can use them as pseudo-states

TT29594
sofien-moalla and others added 16 commits May 15, 2024 08:26
Currently translated at 100.0% (352 of 352 strings)

Translation: rma-16.0/rma-16.0-rma
Translate-URL: https://translation.odoo-community.org/projects/rma-16-0/rma-16-0-rma/es/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: rma-16.0/rma-16.0-rma
Translate-URL: https://translation.odoo-community.org/projects/rma-16-0/rma-16-0-rma/
Currently translated at 90.9% (320 of 352 strings)

Translation: rma-16.0/rma-16.0-rma
Translate-URL: https://translation.odoo-community.org/projects/rma-16-0/rma-16-0-rma/it/
Currently translated at 90.9% (320 of 352 strings)

Translation: rma-16.0/rma-16.0-rma
Translate-URL: https://translation.odoo-community.org/projects/rma-16-0/rma-16-0-rma/it/
Currently translated at 90.9% (320 of 352 strings)

Translation: rma-16.0/rma-16.0-rma
Translate-URL: https://translation.odoo-community.org/projects/rma-16-0/rma-16-0-rma/it/
Currently translated at 48.2% (170 of 352 strings)

Translation: rma-16.0/rma-16.0-rma
Translate-URL: https://translation.odoo-community.org/projects/rma-16-0/rma-16-0-rma/fr/
Currently translated at 100.0% (352 of 352 strings)

Translation: rma-16.0/rma-16.0-rma
Translate-URL: https://translation.odoo-community.org/projects/rma-16-0/rma-16-0-rma/it/
Included finished state to discard those manually finished RMAs

TT47471
Currently translated at 100.0% (352 of 352 strings)

Translation: rma-16.0/rma-16.0-rma
Translate-URL: https://translation.odoo-community.org/projects/rma-16-0/rma-16-0-rma/it/
@peluko00
Copy link
Author

I still see commits to squash:

imagen

Done!

Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

There's still commits to squash, but let's not be more picky this time.

/ocabot merge nobump

@peluko00
Copy link
Author

There's still commits to squash, but let's not be more picky this time.

/ocabot merge nobump

Seems like runboat no detect the command

@pedrobaeza
Copy link
Member

This is not runboat, but a separate agent. It works with a queue, and during nights, it performs a lot of jobs that are still not completed, so it usually takes till 9:00/9:30 to start processing the "today" commands.

@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). 🤖

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 17.0-ocabot-merge-pr-385-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit a149654 into OCA:17.0 May 15, 2024
7 checks passed
@OCA-git-bot
Copy link
Contributor

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

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.