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

[12.0][FIX] l10n_nl_bsn: searching on BSN without dots #232

Merged
merged 2 commits into from
Dec 11, 2020

Conversation

astirpe
Copy link
Member

@astirpe astirpe commented May 23, 2019

Module l10n_nl_bsn reformats the BSN by placing dots in the number.
This PR allows the searching on a number even without specifying the dots.

Fixes #231 for V12.

@astirpe
Copy link
Member Author

astirpe commented May 23, 2019

@thomaspaulb I'm hereby experimenting the searching on BSN without dots. I'm sure the solution can be refined and improved but for the moment this is a quick fix. Would you give it a look?

@astirpe astirpe marked this pull request as ready for review May 23, 2019 08:48
l10n_nl_bsn/models/res_partner.py Show resolved Hide resolved
l10n_nl_bsn/readme/ROADMAP.rst Show resolved Hide resolved
@astirpe astirpe force-pushed the 12_fix_l10n_nl_bsn branch 2 times, most recently from b81e015 to df0c4d3 Compare July 31, 2020 10:23
@NL66278
Copy link
Contributor

NL66278 commented Dec 11, 2020

/ocabot merge minor

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 12.0-ocabot-merge-pr-232-by-NL66278-bump-minor, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit e47900e into OCA:12.0 Dec 11, 2020
@OCA-git-bot
Copy link
Contributor

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

@astirpe astirpe deleted the 12_fix_l10n_nl_bsn branch December 11, 2020 21:33
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.

None yet

5 participants