search

Inxmail / inx_magento1

Inxmail Professional Email Marketing für Magento CE und EE (1.x)
0 stars 0 forks source link

dynamisches Attribut-Mapping #5

Closed falco-knapp closed 9 years ago

falco-knapp commented 9 years ago

Die Detaillierte Beschreibung des Change-Request ist in _ChangeRequest_02_InxmailMagentoExtension_v1.1Flagbit.pdf beschrieben.

falco-knapp commented 9 years ago

Frage: In Magento kann ein Benutzer mehrere Adresse besitzen. Auf welcher dieser Adressen soll sich das Mapping beziehen?

Antwort: Das Mapping soll sich auf die Standard-Rechnungsadresse beziehen.

BrocksiNet commented 9 years ago

Das dynamische Attribut-Mapping wurde in Branch INX-4 umgesetzt.

Siehe: https://github.com/Inxmail/inx_magento1/commits/INX-4

falco-knapp commented 9 years ago

Beobachtungen:

  1. Anmeldung über Account => ja, aber DOI wurde 2 mal ausgelöst (doppelte Anmeldung). Konnte nicht wieder reproduziert werden
  2. Anmeldedatum an Liste wird mit jedem Sync überschrieben (dadurch sind u.U. nachfolgekampagnen nicht steuerbar) (dieses Problem existiert bereits in früheren Versionen)
  3. => NOK. Weder in den Logfiles noch in der Admin-Oberfläche ist ersichtlich, dass sich am Mapping etwas geändert hat (also, wenn ein bestehendes Mapping nicht mehr funktioniert)

Folgende Tests habe ich durchgeführt:

  1. Installation des Moduls in Madison Island (Standard) => OK
  2. Installation des Moduls auf vorgängerversion (3.1.0.4) => OK
  3. Registrierung account + Newsletter => ja, aber DOI wurde 2 mal ausgelöst (doppelte anmeldung) => konnte ich nicht nochmal reproduzieren.
  4. Synchronisation erzwingen und übertragene Felder überprüfen => hat geklappt.
  5. Neue Felder in Inxmail hinzufügen (z.B. Straße) und Mapping anpassen: Wurden entsprechend angezeigt.
  6. Synchronisation und Daten prüfen => wurden korrekt übertragen
  7. Anmeldung guest => ja, anmeldung im Protokoll nur 1 mal.
  8. Abmeldung Account => Hat funktioniert.
  9. Abmeldung guest => hat funktioniert

Acceptance Criteria:

  1. The user can add mappings between customer-attributes in Magento and Inxmail collumns. => OK
  2. Each mapping consists of a source attribute (the Magento-attribute) and a target attribute (Inxmail collumn). => OK
  3. The source attribute can be selected from a dropdown box. => OK
  4. The source attribute can be any attribute that is available on the Magento customer object. => seems ok
  5. If custom attributes have been added to the Magento customer-object, these attributes shall be available in the drop-down as well. How to test?
  6. The target attribute shall be selectable from a dropdown box in similar way as it is currently feasible. => OK
  7. The user is able to see all mappings done so far. => OK
  8. The synchronisation functionality uses this mapping to synchronise the corresponding data. => OK
  9. The dynamic mapping replaces the static mapping described in the summary above. => OK
  10. If an upgrade of the module is undertaken on an existing installation, a migration of the existing mapping should be done to the new structure. After the upgrade, the synchronisation should work as previously with the old mapping. => OK
  11. On a new installation, there shall be no preconfigured mapping. => OK
  12. Other functionality shall not be modified without notification and coordination with Inxmail. => Seems OK.
  13. In order to update the installation- and configuration manual, the appropriate Information is provided together with descriptive screenshots. Based on this information, Inxmail is going to update the documentation. => Fehlt.
  14. If a customer attributes is not available any longer but the mapping configuration is still available, it shall be ignored during synchronisation. Corresponding values in Inxmail shall be left unchanged (there shall be no transmission of empty values). The same applies to attributes in Inxmail that are not existing any longer but are still referenced in the mapping. => keine Fehlermeldung, Daten werden synchronisiert => OK
  15. If a customer attribute in Magento or in Inxmail is no longer available, the error will arise during synchronisation and a protocol message shall be generated. If possible, this message shall be displayed to the administrator once he logs into the backoffice application. => NOK. Weder in den Logfiles noch in der Admin-Oberfläche ist ersichtlich, dass sich am Mapping etwas geändert hat.
  16. The only mandatory attribute to be synchronised is the e-mail address. All other attributes are optional. => OK.
falco-knapp commented 9 years ago

Offen:

  1. If a customer attribute in Magento or in Inxmail is no longer available, the error will arise during synchronisation and a protocol message shall be generated. If possible, this message shall be displayed to the administrator once he logs into the backoffice application. => NOK. Weder in den Logfiles noch in der Admin-Oberfläche ist ersichtlich, dass sich am Mapping etwas geändert hat und daher unvollständig ist. Das erschwert die Fehleranalyse.
janineobert commented 9 years ago

developer added logging and admin notification

falco-knapp commented 9 years ago

Hallo Frau Obert,

damit wir das Testen können, müsste der Code entsprechend auch committed werden und ein neues Release erstellt werden. Können Sie dies bitte veranlassen?

Vielen Dank und Gruß,

Falco Knapp

P.S. Ich würde vorschlagen auf das "Sie" generell zu verzichten und auf "Du" umzuschwenken wäre dies auch in Ihrem Sinne? :)

janineobert commented 9 years ago

Hallo Falco,

gerne per DU ;-)

Habe es veranlasst und gebe dir gleich Rückmeldung sobald ich sie vim Entwickler habe.

Gruß

Janine

janineobert commented 9 years ago

ist zum Testen bereit...

falco-knapp commented 9 years ago

Vielen Dank. Habe es getestet. Scheint alles soweit zu klappen. Prima :)