Skip to content

12.0 mig base_search_fuzzy #1478

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
Merged
pedrobaeza merged 13 commits into from
Jan 14, 2019
Merged

12.0 mig base_search_fuzzy #1478

pedrobaeza merged 13 commits into from
Jan 14, 2019

Conversation

@ernestotejeda
Copy link
Member

@ernestotejeda ernestotejeda commented Jan 10, 2019

Christoph Giesel and others added 3 commits January 10, 2019 02:51
@ernestotejeda
Renamed module base_trgm_search to base_search_fuzzy, added Unit test…
85acb5e 
…s, added translations, added access permissions, moved the monkey patching to method _register_hook of ir.model and fixed _auto_init, added README, cleaned up some aprts
@dreispt @ernestotejeda
Fix typo
741c2be
@oca-transbot @ernestotejeda
OCA Transbot updated translations from Transifex
50d49ed
@ernestotejeda ernestotejeda mentioned this pull request Jan 10, 2019
Merged
@pedrobaeza pedrobaeza added this to the 12.0 milestone Jan 10, 2019
@OCA-git-bot OCA-git-bot mentioned this pull request Jan 10, 2019
Closed
32 tasks
@pedrobaeza
Copy link
Member

pedrobaeza commented Jan 10, 2019

Please remove the warning:

2019-01-10 15:51:21,489 152 WARNING openerp_test odoo.models: The model trgm.index has no _description

pedrobaeza
pedrobaeza reviewed Jan 10, 2019
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.

Please squash together these commit for enhancing commit history:

seleccion_024

@ernestotejeda ernestotejeda force-pushed the 12.0-mig-base_search_fuzzy branch from 4a1100e to 709a7b3 Compare January 10, 2019 20:41
@ernestotejeda ernestotejeda force-pushed the 12.0-mig-base_search_fuzzy branch from 709a7b3 to cf88200 Compare January 10, 2019 20:49
cubells
cubells approved these changes Jan 14, 2019
View reviewed changes
Copy link
Member

@cubells cubells left a comment

Choose a reason for hiding this comment

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

👍

LGTM

Code review and test

@pedrobaeza pedrobaeza merged commit 5902af6 into OCA:12.0 Jan 14, 2019
@pedrobaeza pedrobaeza deleted the 12.0-mig-base_search_fuzzy branch January 14, 2019 12:22
SiesslPhillip pushed a commit to grueneerde/OCA-server-tools that referenced this pull request Nov 20, 2024
@bt-admin
Merge pull request OCA#1478 from OCA/14.0
479db50 
Syncing from upstream OCA/server-tools (14.0)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pedrobaeza pedrobaeza pedrobaeza approved these changes

+1 more reviewer

@cubells cubells cubells approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

approved needs review

Projects

None yet

Milestone

12.0

Development

Successfully merging this pull request may close these issues.

10 participants

@ernestotejeda @pedrobaeza @cubells @OCA-git-bot @dreispt @oca-transbot @serpentcs-dev1 @lasley @gdgellatly @oca-travis