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

[13.0][MIG] l10n_ch_base_bank - reduced #550

Merged
merged 140 commits into from Dec 9, 2020
Merged

[13.0][MIG] l10n_ch_base_bank - reduced #550

merged 140 commits into from Dec 9, 2020

Conversation

yvaucher
Copy link
Member

@yvaucher yvaucher commented Jun 4, 2020 •
edited

nbessi and others added 30 commits April 2, 2020 11:02
@nbessi
[IMP] refactoring module structure
86741d1
@nbessi
[FIX] port of base bank
520b872
@nbessi
[REF] bvr report
cca6228
@nbessi
[REF] bvr duplicated stuff
2b45f51
@nbessi
[REF] move from spli to bank"
a23da7e
@nbessi
[FIX] non serializable type
c9d2bcc
@nbessi
[FIX] default in sale workflow fo generated invoices
39736e1
@nbessi
[TYPO]
2ae1cd3
@nbessi
[REFACTOR] bank dispertion
153477a
@nbessi
[FIX] contributors list
e8f9913
@nbessi
[FIX] encoding metadata
244b5b5
@nbessi
[ADD] add post related field
24ae47a
@nbessi
[IMP] add notion of CCP on bank instead of having an account on res.p…
9a343e3 
…artner.bank.

The code is retro compatible and if you do not have a ccp on bank it will act as before
@nbessi
[IMP] change version of addons + update changelog
f1d0cb6
@nbessi
[FIX] res.partner.bank view
c69d22c
@nbessi
[FIX] ccp related is read only
f812b78
@nbessi
[FIX] blank line
e226dd3
@nbessi
[FIX] bank view city and zip where duplicated
c82000a
@nbessi
[FIX] hide ccp if the bic is the postfinance one
3a38bec
@nbessi
[FIX] bank search and name get. Add a intelligent name search and add…
a5514db 
… city to name get
@nbessi
[ADD] street on res.bank name_get and name_search
fa640d4
@vrenaville
[FIX]only bvr requiered bvr ref
bf6768a
@jgrandguillaume
[IMP] Add all translations .pot file in order to allow people to tran…
438870a 
…slate module on LP
@guewen
fix a bug when an account number is empty
2d8f068
Launchpad automatic translations update.
751b584
@nbessi
Set all addons to uninstallable
7471622
@nbessi
Rename all addons to xxx_unported
24bc316
@PCatinean
[ADD] Added l10n_ch_base_bank migrated to the new api
ec23f6f
@nbessi
Cleanup of l10n_ch_base_bank port
7541b70
@nbessi
Remove deprecated size
a739ae4
@yvaucher
Fix tests by desambiguating ISR subscr and postal
4f43987 
An ISR subscription number is a reserved range of postal account to generate ISR
It must start with 01 or 03

Use more realistic data in tests
@yvaucher
Fix removed view
a5340a3
@yvaucher
Fix test set empty M2O with empty recordset
262d126
@yvaucher
Clearing do not have leading 0s
e878ad9
@yvaucher
[REF] l10n_ch_postal onchanges
5613606 
Replace "Postal number" by "ISR"
Detect an ISR subscription number based on the leading digits 01 or 03
@yvaucher
Document
045bf1c
@yvaucher
Signature change of _is_isr_ref
eee7bca
@yvaucher
Adapt to odoo core change that removed _is_isr_ref
8fda33c 
Follow the removal from here: odoo/odoo#53815
@yvaucher yvaucher force-pushed the 13-l10n_ch_base_bank-reduced branch from d0bcfb4 to 8fda33c Compare August 13, 2020 15:16
@yvaucher yvaucher mentioned this pull request Aug 14, 2020
Closed
@yvaucher yvaucher mentioned this pull request Sep 16, 2020
Closed
2 tasks
@gurneyalex
Copy link
Member

@yvaucher what is the status of this PR? the last PR on Odoo is closed, and you say it's ok, but #578 is red -> how can we move on here?

@yvaucher
Copy link
Member Author

yvaucher commented Dec 8, 2020

@gurneyalex AFAIK this module has been used since a while on our projects in v13

@yvaucher
Copy link
Member Author

yvaucher commented Dec 8, 2020

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 13.0-ocabot-merge-pr-550-by-yvaucher-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Dec 8, 2020
@OCA-git-bot
Merge PR #550 into 13.0
146006a 
Signed-off-by yvaucher
@TDu
Copy link
Member

TDu commented Dec 8, 2020

Yes thank you. Go 🤖

@TDu
Copy link
Member

TDu commented Dec 9, 2020

@yvaucher Looks like the merge failed because of duplicate in the .po file https://travis-ci.com/github/OCA/l10n-switzerland/jobs/456034459#L914
Would be great if you could fix and try merging again, thanks.

@yvaucher
Copy link
Member Author

yvaucher commented Dec 9, 2020

@TDu thanks for the pointer

@yvaucher
Copy link
Member Author

yvaucher commented Dec 9, 2020

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 13.0-ocabot-merge-pr-550-by-yvaucher-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit a40c767 into OCA:13.0 Dec 9, 2020
@OCA-git-bot
Copy link
Contributor

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

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 13.0-ocabot-merge-pr-550-by-yvaucher-bump-nobump, awaiting test results.

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

@TDu TDu TDu approved these changes

Assignees
No one assigned
Labels
bot is merging ⏳ merged 🎉
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet