[13.0][MIG] sale_report_country_state: Migration to 13.0 #112
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
joao-p-marques
force-pushed
the
13.0-mig-sale_report_country_state
branch
from
ec16ad8
to
62ff243
Compare
pedrobaeza
approved these changes
May 25, 2021
victoralmau
suggested changes
May 25, 2021
| { | ||
| "name": "Sale Report Filter by State", | ||
| "version": "13.0.1.0.0", | ||
| "author": "Tecnativa," "Odoo Community Association (OCA)", |
There was a problem hiding this comment.
Suggested change
| "author": "Tecnativa," "Odoo Community Association (OCA)", | |
| "author": "Tecnativa, Odoo Community Association (OCA)", |
sale_report_country_state/i18n/es.po
Outdated
Comment on lines
35
to
40
|
|
||
| #~ msgid "All sales orders grouped by sales channels" | ||
| #~ msgstr "Todos los pedidos de venta agrupados por canal" | ||
|
|
||
| #~ msgid "Sales Orders Statistics" | ||
| #~ msgstr "Estadísticas de pedidos de ventas" |
There was a problem hiding this comment.
Suggested change
| #~ msgid "All sales orders grouped by sales channels" | |
| #~ msgstr "Todos los pedidos de venta agrupados por canal" | |
| #~ msgid "Sales Orders Statistics" | |
| #~ msgstr "Estadísticas de pedidos de ventas" |
chienandalu
suggested changes
May 25, 2021
sale_report_country_state/README.rst
Outdated
Comment on lines
43
to
44
| #. Go to *Sales > Reporting > All Channels Sales Orders* | ||
| #. Filter by State. |
| _inherit = "report.all.channels.sales" | ||
|
|
||
| state_id = fields.Many2one( | ||
| comodel_name="res.country.state", string="Partner's State", readonly=True, |
There was a problem hiding this comment.
Change to Customer State so it's grouped altogether with the other partner's properties
joao-p-marques
force-pushed
the
13.0-mig-sale_report_country_state
branch
from
62ff243
to
1607bb6
Compare
|
Thanks @chienandalu @victoralmau Changes done! |
victoralmau
approved these changes
May 25, 2021
chienandalu
approved these changes
May 25, 2021
|
/ocabot merge nobump |
|
Hey, thanks for contributing! Proceeding to merge this for you. |
|
Congratulations, your PR was merged at 6871c49. 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
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
@Tecnativa
TT27872
@chienandalu @pedrobaeza can you review please?