What are the Slack Archives?

It’s a history of our time together in the Slack Community! There’s a ton of knowledge in here, so feel free to search through the archives for a possible answer to your question.

Because this space is not active, you won’t be able to create a new post or comment here. If you have a question or want to start a discussion about something, head over to our categories and pick one to post in! You can always refer back to a post from Slack Archives if needed; just copy the link to use it as a reference..

Hey guys, I just updated sprkyer/sales from `11.34.1` to `11.34.3`, now I get the problem, that runn

michael.schmetter
michael.schmetter Posts: 119 🧑🏻‍🚀 - Cadet

Hey guys,
I just updated sprkyer/sales from 11.34.1 to 11.34.3, now I get the problem, that running docker/sdk console transfer:databuilder:generate fails with:

Exception - Exception: Value mismatch for "Item.sku" transfer property. Value1: "isbn10"; Value2: "isbn10()". To fix this, search for 'property name="sku"' in the code base and fix the wrong one. 

In 11.34.2 there was a change to add () to prevent deprecation messages:
https://github.com/spryker/sales/commit/baf1a79b2fff1394d196eb5b874a0dacee06d6af
That was part of that release group:
https://api.release.spryker.com/release-group/4535

What am I missing here, that that command is failing?

Comments

  • U01A5ARAXP0
    U01A5ARAXP0 Posts: 119 🧑🏻‍🚀 - Cadet

    You probably have some project level override on the databuilder for the sku property with isbn10, while on spryker level now its isbn10()

  • michael.schmetter
    michael.schmetter Posts: 119 🧑🏻‍🚀 - Cadet

    Ok, issue is actually, that the other changes of that release group also need to be updated. If not I get the error above, because some modules are setting it with, some without the ()

  • michael.schmetter
    michael.schmetter Posts: 119 🧑🏻‍🚀 - Cadet

    For anyone who comes also here having that problem:
    https://github.com/spryker/user/releases/tag/3.14.1
    That is missing in the release group