Skip to content

8.0 product public category partner - #4

Closed
foutoucour wants to merge 3 commits into
OCA:8.0from
savoirfairelinux:8.0-product_public_category_partner
Closed

foutoucour wants to merge 3 commits into
OCA:8.0from
savoirfairelinux:8.0-product_public_category_partner

Conversation

@foutoucour

Copy link
Copy Markdown

Simple module that add a relation between product.public.category and res.partner.

It was used to be able to make a domain to filter product.public.category by partner.

@pedrobaeza

Copy link
Copy Markdown
Member

Why many2one instead a many2many? Are you only enabling a product to one partner? I see also that functionally speaking, users expect the contrary: select which products a partner can see in the partner form, not going to product to select the partner.

@rafaelbn

rafaelbn commented Sep 1, 2015

Copy link
Copy Markdown
Member

Please @foutoucour could you answer @pedrobaeza ? If not interested please close PR. Thanks

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

move this to README.rst and use the OCA template

@hbrunn hbrunn added this to the 8.0 milestone Nov 20, 2015
@rafaelbn

Copy link
Copy Markdown
Member

Hi @hbrunn I'm sorry for you time in this PR, author doesn't touch it from 5 Dec 2014. I close.

@rafaelbn rafaelbn closed this Nov 21, 2015
cubells pushed a commit to Tecnativa/website that referenced this pull request Nov 11, 2016
bealdav pushed a commit to akretion/website that referenced this pull request Feb 10, 2017
Add base files for the branch (.travis.yml, .coveragerc, .gitignore, README.md, LICENSE)
zamberjo pushed a commit to aurestic/website that referenced this pull request Nov 15, 2019
SirAionTech added a commit to SirAionTech/website that referenced this pull request Dec 9, 2024
edescalona added a commit to BinhexTeam/website that referenced this pull request Dec 24, 2024
# This is the 1st commit message:

[IMP] Add tests

# This is the commit message OCA#2:

[IMP] Add tests

# This is the commit message OCA#3:

[FIX] Tests

# This is the commit message OCA#4:

[FIX] Tests

[ADD] Tests

[FIX] Tests

[FIX] Tests

[ADD] Tests

[ADD] Tests

[FIX] Tests

[ADD] Tests

[ADD] Tests

[REM] Tests

[ADD] Tests

[ADD] Tests

[ADD] Tests
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.

4 participants