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.

Sign up for GitHub

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

Jump to bottom

[17.0][MIG] account_statement_import_sheet_file: Migration to 17.0 #654

Merged
merged 45 commits into from
May 21, 2024
Merged

[17.0][MIG] account_statement_import_sheet_file: Migration to 17.0 #654

merged 45 commits into from
May 21, 2024

Conversation

rov-adhoc
Copy link

@rov-adhoc rov-adhoc force-pushed the 17.0-mig-account_statement_import_sheet_file branch 5 times, most recently from f6f562b to 28fa47e Compare December 28, 2023 18:25
@xtanuiha
Copy link

xtanuiha commented Apr 9, 2024 •
edited
Loading

I have contributed translations for Simplified Chinese, could you please merge them to your code, thank you! @rov-adhoc

zh_CN.zip

@pedrobaeza
Copy link
Member

That should be added on Weblate once the module is merged.

@rov-adhoc rov-adhoc force-pushed the 17.0-mig-account_statement_import_sheet_file branch from 28fa47e to d5f12fc Compare April 23, 2024 23:41
@rov-adhoc rov-adhoc force-pushed the 17.0-mig-account_statement_import_sheet_file branch 4 times, most recently from 16d9e3a to 0dbf4d4 Compare May 7, 2024 19:59
@pedrobaeza
Copy link
Member

To add: #693

@rov-adhoc rov-adhoc force-pushed the 17.0-mig-account_statement_import_sheet_file branch from 0dbf4d4 to 8010d05 Compare May 13, 2024 13:23
@peluko00
Copy link

Could you continue @rov-adhoc ?

@rov-adhoc rov-adhoc force-pushed the 17.0-mig-account_statement_import_sheet_file branch from 8010d05 to 4691e4a Compare May 15, 2024 12:36
primes2h and others added 14 commits May 15, 2024 09:39
@primes2h @rov-adhoc
[MIG] account_bank_statement_import_txt_xlsx: Migration to 14.0
a5b3734 
[FIX] Mostly currency-related fixes

[IMP] sample_statement_en test: add dates on description

Co-authored-by: Marco Colombo <[email protected]>

[UPD] Update account_statement_import_txt_xlsx.pot
@primes2h @rov-adhoc
Added translation using Weblate (Italian)
74fbf5d 
Translated using Weblate (Italian)

Currently translated at 64.3% (56 of 87 strings)

Translation: bank-statement-import-14.0/bank-statement-import-14.0-account_statement_import_txt_xlsx
Translate-URL: https://translation.odoo-community.org/projects/bank-statement-import-14-0/bank-statement-import-14-0-account_statement_import_txt_xlsx/it/

Translated using Weblate (Italian)

Currently translated at 73.5% (64 of 87 strings)

Translation: bank-statement-import-14.0/bank-statement-import-14.0-account_statement_import_txt_xlsx
Translate-URL: https://translation.odoo-community.org/projects/bank-statement-import-14-0/bank-statement-import-14-0-account_statement_import_txt_xlsx/it/

Translated using Weblate (Italian)

Currently translated at 98.8% (86 of 87 strings)

Translation: bank-statement-import-14.0/bank-statement-import-14.0-account_statement_import_txt_xlsx
Translate-URL: https://translation.odoo-community.org/projects/bank-statement-import-14-0/bank-statement-import-14-0-account_statement_import_txt_xlsx/it/
@pedrocs-exo @rov-adhoc
Added translation using Weblate (Portuguese)
9316f0a
@primes2h @rov-adhoc
Translated using Weblate (Italian)
990750b 
Currently translated at 100.0% (87 of 87 strings)

Translation: bank-statement-import-14.0/bank-statement-import-14.0-account_statement_import_txt_xlsx
Translate-URL: https://translation.odoo-community.org/projects/bank-statement-import-14-0/bank-statement-import-14-0-account_statement_import_txt_xlsx/it/
@JordiBForgeFlow @rov-adhoc
[account_statement_import_txt_xlsx][imp] add migration scripts
d5df013
@ibuioli @rov-adhoc
Added translation using Weblate (Spanish (Argentina))
d66319e
@ibuioli @rov-adhoc
Translated using Weblate (Spanish (Argentina))
96a2fd5 
Currently translated at 100.0% (87 of 87 strings)

Translation: bank-statement-import-14.0/bank-statement-import-14.0-account_statement_import_txt_xlsx
Translate-URL: https://translation.odoo-community.org/projects/bank-statement-import-14-0/bank-statement-import-14-0-account_statement_import_txt_xlsx/es_AR/

account_statement_import_txt_xlsx 14.0.1.0.1
@nicomacr @rov-adhoc
[MIG] account_statement_import_txt_xlsx: Migration to 15.0
8eeb643 
[UPD] Update account_statement_import_txt_xlsx.pot

[UPD] README.rst
@francesco-ooops @rov-adhoc
Translated using Weblate (Italian)
d6f5f09 
Currently translated at 100.0% (87 of 87 strings)

Translation: bank-statement-import-15.0/bank-statement-import-15.0-account_statement_import_txt_xlsx
Translate-URL: https://translation.odoo-community.org/projects/bank-statement-import-15-0/bank-statement-import-15-0-account_statement_import_txt_xlsx/it/
@rov-adhoc
[FIX] account_bank_statement_import_txt_xlsx: import sheet notes
2683695
@norlinhenrik @rov-adhoc
[IMP] account_statement_import_txt_xlsx: wizard: 2 amount columns, fo…
ddcd036 
…r IN and OUT

[IMP] account_statement_import_txt_xlsx: wizard: amount2_reverse boolean field, with tests
@norlinhenrik @rov-adhoc
[IMP] account_statement_import_txt_xlsx: amount_debit_column, amount_…
9d59ac1 
…credit_column
@JordiBForgeFlow @rov-adhoc
[IMP] account_statement_import_txt_xlsx: Allow mapping with reference…
8b33c8b 
… to column numbers, and concatenation

A new field 'File does not contain header line' is added in the Statement Sheet Mappings. If you set to True,
then you can map the columns by indicating in each field of the 'Columns' section the column number in the file.

We also allow to concatenate multiple columns in the file to a single column to a single field of the statement line.
You have to indicate the names of the columns separated by comma.
@JordiBForgeFlow @rov-adhoc
[IMP] account_statement_import_txt_xlsx: add tests
a9a377f
Ömer KÜLAK and others added 15 commits May 15, 2024 09:39
@rov-adhoc
Translated using Weblate (Turkish)
4c3b6f4 
Currently translated at 95.9% (95 of 99 strings)

Translation: bank-statement-import-16.0/bank-statement-import-16.0-account_statement_import_sheet_file
Translate-URL: https://translation.odoo-community.org/projects/bank-statement-import-16-0/bank-statement-import-16-0-account_statement_import_sheet_file/tr/
@Ivorra78 @rov-adhoc
Translated using Weblate (Spanish)
20ce381 
Currently translated at 100.0% (99 of 99 strings)

Translation: bank-statement-import-16.0/bank-statement-import-16.0-account_statement_import_sheet_file
Translate-URL: https://translation.odoo-community.org/projects/bank-statement-import-16-0/bank-statement-import-16-0-account_statement_import_sheet_file/es/
@mymage @rov-adhoc
Translated using Weblate (Italian)
53812bb 
Currently translated at 58.5% (58 of 99 strings)

Translation: bank-statement-import-16.0/bank-statement-import-16.0-account_statement_import_sheet_file
Translate-URL: https://translation.odoo-community.org/projects/bank-statement-import-16-0/bank-statement-import-16-0-account_statement_import_sheet_file/it/
@alexis-via @rov-adhoc
[IMP] account_statement_import_sheet_file: add quote as thousand sepa…
7801ea3 
…rator
@alessandro-fiorino @rov-adhoc
Translated using Weblate (Italian)
3cc3fe9 
Currently translated at 98.9% (98 of 99 strings)

Translation: bank-statement-import-16.0/bank-statement-import-16.0-account_statement_import_sheet_file
Translate-URL: https://translation.odoo-community.org/projects/bank-statement-import-16-0/bank-statement-import-16-0-account_statement_import_sheet_file/it/
@rov-adhoc
[UPD] Update account_statement_import_sheet_file.pot
2b35525
@OCA-git-bot @rov-adhoc
[BOT] post-merge updates
f902adc
@weblate @rov-adhoc
Update translation files
3ca0990 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: bank-statement-import-16.0/bank-statement-import-16.0-account_statement_import_sheet_file
Translate-URL: https://translation.odoo-community.org/projects/bank-statement-import-16-0/bank-statement-import-16-0-account_statement_import_sheet_file/
@mymage @rov-adhoc
Translated using Weblate (Italian)
a438a7d 
Currently translated at 100.0% (100 of 100 strings)

Translation: bank-statement-import-16.0/bank-statement-import-16.0-account_statement_import_sheet_file
Translate-URL: https://translation.odoo-community.org/projects/bank-statement-import-16-0/bank-statement-import-16-0-account_statement_import_sheet_file/it/
@mileo @rov-adhoc
[IMP][account_statement_import_sheet_file] Empty lines with spaces, o…
15f9b13 
…r with simbols eg: USD/$
@berisoft-arg @rov-adhoc
Translated using Weblate (Spanish)
ba7cbb7 
Currently translated at 100.0% (100 of 100 strings)

Translation: bank-statement-import-16.0/bank-statement-import-16.0-account_statement_import_sheet_file
Translate-URL: https://translation.odoo-community.org/projects/bank-statement-import-16-0/bank-statement-import-16-0-account_statement_import_sheet_file/es/
@mileo @rov-adhoc
[REF][account_statement_import_sheet_file] Remove decimal
a119d82
@OCA-git-bot @rov-adhoc
[BOT] post-merge updates
b20b06c
@rov-adhoc
[IMP] account_statement_import_sheet_file: pre-commit execution
0967843
@rov-adhoc
[MIG] account_statement_import_sheet_file: Migration to 17.0
3566322
@rov-adhoc rov-adhoc force-pushed the 17.0-mig-account_statement_import_sheet_file branch from 4691e4a to 3566322 Compare May 15, 2024 12:40
@rov-adhoc
Copy link
Author

@peluko00 done

@peluko00
Copy link

@peluko00 done

Thanks!

peluko00
peluko00 approved these changes May 21, 2024
View reviewed changes
Copy link

@peluko00 peluko00 left a comment

Choose a reason for hiding this comment

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

Works good, tested in runboat. LGTM

javierobcn
javierobcn approved these changes May 21, 2024
View reviewed changes
Copy link

@javierobcn javierobcn left a comment

Choose a reason for hiding this comment

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

Running

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

pedrobaeza
pedrobaeza approved these changes May 21, 2024
View reviewed changes
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.

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

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

@OCA-git-bot OCA-git-bot merged commit 0d08f88 into OCA:17.0 May 21, 2024
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 55cd6b0. 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
Reviewers

@peluko00 peluko00 peluko00 approved these changes

@pedrobaeza pedrobaeza pedrobaeza approved these changes

@javierobcn javierobcn javierobcn approved these changes

Assignees
No one assigned
Labels
approved merged 🎉 ready to merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.