-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
B2B shared catalog feature set #28550
Comments
Hi @nrkapoor. Thank you for your report.
Please make sure that the issue is reproducible on the vanilla Magento instance following Steps to reproduce. To deploy vanilla Magento instance on our environment, please, add a comment to the issue:
For more details, please, review the Magento Contributor Assistant documentation. Please, add a comment to assign the issue:
|
Hi @dani97. Thank you for working on this issue.
|
Hi @pmarjan. Thank you for working on this issue.
|
upon closer inspection of the related PRs to this ticket, I noticed that there is NO new schema that is specific to the shared catalog. For example the schema.graphqls that has been added to the CatalogPermissionsGraphQl module does only extend two queries
and they are both specific to the catalog. Thus no wonder, most of the coding relates to Catalog Permissions and can be found here: In attempt to put focus on differences and scope, I will further say that the title of this PR is inaccurate: catalog permissions are a thing on its own - and NOT only for the shared catalog. So, can you please confirm: we do not have shared catalog specific graphql schema? If so, probably it will be more clear separation of concerns to have specified tickets like: a. GraphQL "categoryList" query does not consider Category Permissions configuration Because, I think this is exactly what is missing and the PR tries to solve. Also it will be consistent continuation of an earlier work on the same topic: GraphQL product search does not consider Category Permissions configuration |
@pmarjan it has to also support products query. |
As a Magento Developer,
I want to expose B2B shared catalog functionality via GraphQL
So that customers can use a more flexible and performant API technology.
Connected tickets
Acceptance Criteria
GraphQL respects user permissions for all storefront operations.
WebAPI test coverage and indirectly covered by its sub-tickets
All builds are green
Related Pull Requests
https://github.com/magento/partners-magento2b2b/pull/335
and is continuation from PR
https://github.com/magento/partners-magento2b2b/pull/96
Categories permissions for "categories" and "categoryList" #29883
https://github.com/magento/partners-magento2ee/pull/319
magento/magento2#29926: Prices should be possibly hidden from products query results #29998
Search should be disabled from products query when general configuration chooses to disabled it #30118
https://github.com/magento/partners-magento2ee/pull/337
Add to Cart mutations in GraphQl should consider Catalog Permissions #30019
https://github.com/magento/partners-magento2ee/pull/328
Other relater PRs are
Extract getting customer group id from context into its own class
https://github.com/magento/partners-magento2ee/pull/326
Increase accuracy in the context isCustomer method
Context "isCustomer" should return false for Admin and Integration tokens #29894
The text was updated successfully, but these errors were encountered: