search

OCA / multi-company

GNU Affero General Public License v3.0
100 stars 281 forks source link

[14.0] [FIX] base_multi_company: apply '_name_search' fix to 'search_read' #538

Closed renda-dev closed 9 months ago

renda-dev commented 9 months ago

The same patch applied to _name_search could be applied to search_read, to fix some different scenarios (e.g. "Search More..." button in m2x relations would open a tree view that applies search_read and didn't work before this fix).

OCA-git-bot commented 9 months ago

Hi @pedrobaeza, some modules you are maintaining are being modified, check this out!

francesco-ooops commented 9 months ago

@pedrobaeza good to go?

pedrobaeza commented 9 months ago

/ocabot merge patch

You should apply these patches to upper versions to be all paired. If not, when you decide to go to another future version (you seem stuck at v14 for now, which is not very advisable IMO, but everyone follow the path they consider), you will need to analyze again the same problems (memory is short), to finally find that you already fixed this in the past.

OCA-git-bot commented 9 months ago

What a great day to merge this nice PR. Let's do it! Prepared branch 14.0-ocabot-merge-pr-538-by-pedrobaeza-bump-patch, awaiting test results.

OCA-git-bot commented 9 months ago

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

francesco-ooops commented 9 months ago

/ocabot merge patch

You should apply these patches to upper versions to be all paired. If not, when you decide to go to another future version (you seem stuck at v14 for now, which is not very advisable IMO, but everyone follow the path they consider), you will need to analyze again the same problems (memory is short), to finally find that you already fixed this in the past.

agreed, time and budget allowing we'll try to fw-port as much as possible