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

13.0 mig web export view #2278

Closed
wants to merge 56 commits into from
Closed

Conversation

vancouver29
Copy link

[13.0][MIG]web_export_view

Leonardo Pistone and others added 30 commits August 18, 2022 14:49
* Compatibility with Werkzeug 0.9.3
* Export of first one2many field works again
* Parsing of float values
… with the original index used in the native excel export
Monetary fields were being exported empty because the parsing failed.

In the way of correctly exporting them as numbers, this chunk of code's performance has been improved.
* Retrieve all columns even if they are not sortable. Prevent to get select record column
* Prevent to process float time numbers as real numbers during the export
Updated by Update PO files to match POT (msgmerge) hook in Weblate.
OCA-git-bot and others added 23 commits August 18, 2022 14:49
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

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

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

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

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

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

Translation: web-12.0/web-12.0-web_export_view
Translate-URL: https://translation.odoo-community.org/projects/web-12-0/web-12-0-web_export_view/hr/
Currently translated at 25.0% (1 of 4 strings)

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

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

Translation: web-12.0/web-12.0-web_export_view
Translate-URL: https://translation.odoo-community.org/projects/web-12-0/web-12-0-web_export_view/fr/
Currently translated at 50.0% (2 of 4 strings)

Translation: web-12.0/web-12.0-web_export_view
Translate-URL: https://translation.odoo-community.org/projects/web-12-0/web-12-0-web_export_view/tr/
raise AttributeError()
return super(ExcelExportView, self).__getattribute__(name)
return super(ExcelExportView, self).__getattribute__(name)
raise AttributeError()

Choose a reason for hiding this comment

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

Why do you change origin function like this?

Copy link
Author

Choose a reason for hiding this comment

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

@yweng8111 It is just for the purpose of improving Test coverage. The logic stays untouched.

@simahawk
Copy link
Contributor

Please check #1388

image

@simahawk simahawk closed this Aug 24, 2022
@vancouver29
Copy link
Author

vancouver29 commented Aug 24, 2022

Please check #1388

image

@simahawk, I agree that this export function already exists in v13. But with this Export All, we can only export all the records instead of the selected ones. Since not every user wants to export all the records ...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.