Flag, Variante nur mit Bestand importieren

plentymarkets / plentymarkets-shopware-connector

plentymarkets shopware connector plugin

http://www.plentymarkets.eu/shopware/

MIT License

30 stars 51 forks source link

Flag, Variante nur mit Bestand importieren #360

Closed smx-ea closed 6 years ago

smx-ea commented 6 years ago

Q	A
issue or Enhancement	issue	Enhancement
Changelog updated?	no	yes
License	MIT

What's in this PR?

Explain what the changes in this PR do.

Checklist

[ ] Updated CHANGELOG.md to describe BC breaks / deprecations | new feature | bugfix

jochenmanz commented 6 years ago

Um werte aus der Konfiguration zu beziehen gibt es einen fertigen configservice. Jedoch passt für mich die Anpassung nicht zum connector. Besser wäre es deine Anforderung in ein eigenes Plugin auszulagern. Am einfachsten kann das umgesetzt werden wenn du den productResponseParser und möglicherweise auch den variantResponseParser dekorierst. Schau dir dazu am besten einmal unter Components/Bundle die dort gewählte Lösung an. Ähnliches kann einfach in einem eigenen shopware Pluigin umgesetzt werden.

pm-astephen commented 6 years ago

Dieses Feature wurde von plentymarkets in Auftrag gegeben. Arvatis @Pfabeck bitte einmal prüfen

Danke

jochenmanz commented 6 years ago

@smx-ea wenn dann hättest du den ConfigService nutzen und dich nicht quer beet in die TransferObjekte und Handler hängen müssen: https://github.com/plentymarkets/plentymarkets-shopware-connector/pull/385

Pfabeck commented 6 years ago

Moin, bin im Urlaub aber der Configservice sollte genutzt werden. Rest schau ich mir an wenn ich zurück bin.

Pfabeck commented 6 years ago

Pr #385 mit Configservice umgesetzt