Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@nnnnat
Copy link
Contributor

@nnnnat nnnnat commented Jul 23, 2019

Impact: minor
Type: feature

Changes

  • Transformations that happen during publication to Catalog are now centralized into context.mutations.applyCustomPublisherTransforms, which can be overridden by other plugins to customize the transformation process.
  • Created context.mutations.partialProductPublish, which can be used by any plugin to request a partial re-publish (re-transformation) of any product.
  • Updated inventory plugin to use the new partialProductPublish rather than just rerunning its own transformation. This will ensure that other transformations that depend on inventory can also rerun. For example, if prices depend on available inventory, the prices can be updated on a CatalogProduct after every inventory change.

Breaking changes

None

Testing

  1. Verify that normal publish flow still works. Verify that tax, inventory, and price fields are set on the published catalog product.
  2. partialProductPublish is not exposed through GraphQL at this time, but you can create a custom plugin that calls it to verify that it works as expected.

@aldeed aldeed marked this pull request as ready for review July 26, 2019 14:56
@aldeed aldeed changed the title Feat nnnat catalog partial publish Feat: catalog partial publish / rerun transformations Jul 26, 2019
@aldeed aldeed requested review from rosshadden and ticean July 26, 2019 15:04
@cmbirk
Copy link
Contributor

cmbirk commented Jul 30, 2019

@rosshadden @ticean @aldeed can we go ahead and merge this?

@kieckhafer kieckhafer merged commit 8f84dbb into develop Jul 31, 2019
@kieckhafer kieckhafer deleted the feat-nnnat-catalog-partial-publish branch July 31, 2019 17:49
@kieckhafer kieckhafer mentioned this pull request Aug 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants