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

TIP-1145: Add parent filter #10205

Merged
merged 2 commits into from
Jun 11, 2019
Merged

TIP-1145: Add parent filter #10205

merged 2 commits into from
Jun 11, 2019

Conversation

pierallard
Copy link
Contributor

@pierallard pierallard commented Jun 10, 2019 •
edited

Description in the docs PR : https://github.com/akeneo/pim-api-docs/pull/268/files

Q A
Added Specs -
Added legacy Behats -
Added acceptance tests -
Added integration tests -
Changelog updated -
Review and 2 GTM Todo
Micro Demo to the PO (Story only) Todo
Migration script -
Tech Doc -

@pierallard pierallard force-pushed the TIP-1145 branch 2 times, most recently from dbc966c to c9d4033 Compare June 11, 2019 09:53
@pierallard pierallard merged commit 31d288a into master Jun 11, 2019
@pierallard pierallard deleted the TIP-1145 branch June 11, 2019 15:29
@alvinnguyen
Copy link

Any plans to back-port this to v2 guys?

@ahocquard
Copy link
Contributor

@alvinnguyen hello, it is not planned as far I know. I will confirm it with our Product Manager next week.

@NolwennP
Copy link
Contributor

NolwennP commented Jun 24, 2019 •
edited

Hi @alvinnguyen!
I'm the Product Manager in charge of the API.
To complete Alexandre's answer:

  • We do not plan to backport this in 2.0, 2.1 or 2.2 as those versions are not supported by our team anymore.
  • We also do not plan to backport this in 2.3. The reason is, we only perform patches on our previously released version, like the v2.3 is. Patches only contain fixes. Adding a new filter to the product endpoint is not a fix but a feature.

That's why you'll find this new feature in our next minor version of the PIM, the 3.2.
I hope this answer your question!

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

@ahocquard ahocquard ahocquard approved these changes

@mmetayer mmetayer mmetayer approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@pierallard @alvinnguyen @ahocquard @NolwennP @mmetayer