search

selectline-software / selectline-api

Apache License 2.0
24 stars 5 forks source link

DeliveryAddress wird bei Post /Documents nicht eingetragen #191

Open s52512 opened 2 years ago

s52512 commented 2 years ago

Ich habe folgende JSON Struktur verwendet und es wird ein Beleg erzeugt, jedoch wird die DeliveryAddress nicht gesetzt.

{ "KindFlag": "F", "BusinessPartner": { "Address": { "Number": "10004" }, "DeliveryAddress": { "Number" : 12 } } }

auch der Versuch die DeliveryAddress Number als String zu übergeben führt zum selben Ergebnis.

Für Hinweise dazu wäre ich dankbar.

Testumgebung:

{ "Company": "SelectLine Software GmbH", "Product": "SelectLine API", "Description": "REST Service for SelectLine.Erp", "Version": "21.3.9 Build 498625", "BuildDate": "01/28/2022 06:03:58", "IsBackendManagerRunning": true }

SusanneSchadenberg commented 2 years ago

Hallo, vielen Dank für Ihre Nachricht bzw. Info. Leider muss ich das Fehlerverhalten bestätigen. Derzeit können abw. Lieferadressen nur über die "Adressfelder" angelegt werden. zB. "DeliveryAddress": { "FirstName": "", "LastName": "Meier Bau AG", "Company": "Meier Bau AG", "Address": { "Street": "Ahornwegr 14", "City": "Wittenberge", "ZipCode": "19322", "CountryFlag": "DE",

Ihren Fehler habe ich aufgenommen und intern zur weiteren Prüfung/Behebung weitergeleitet.

Viele Grüße aus Magdeburg Susanne

ThePholph commented 2 years ago

Hallo,

nur als kleine Anmerkung. Ich sehe dies nicht als Fehler. Es ist meines Erachtens korrekt, dass wir als Entwickler die Adressdaten füllen. Die Adressdaten / Lieferadressen zu Kunden können ja vorab via API geladen werden.

Es kommt zudem nicht selten vor, dass eine nicht angelegte Adresse Verwendung findet, die auch nicht in die Stammdaten übernommen werden soll (bspw. Baustellen).

Eine REST-API ist kein Frontend Ersatz, das Frontend inkl. Funktionen (bspw. Lieferadresse wählen und übernehmen) ist ja Aufgabe von uns zu entwickeln, wenn nötig.

Vielen Dank.

Viele Grüße

Daniel

Micha-Richter commented 2 years ago

Naja, solange Number als Property in der Struktur enthalten ist, sollte dieser Weg auch funktionieren.. ;) (und wenn ich dadurch Daten in der Übertragung sparen kann, dann nehme ich das auch gern mit)

Es sollte einfach eindeutig sein, finde ich: entweder Number wird angeboten und funktioniert, oder es gibt das Property eben gar nicht.

ThePholph commented 2 years ago

Hallo Susanne,

ihr habt dies als Bug gekennzeichnet. Wie wird der Fehler nun genau spezifiziert? Sollte die Funktion gegeben sein oder wird das Feld demnächst nicht mehr angeboten? Dies hat enorme Relevanz für unsere aktuellen Umsetzungen. Daher wäre es schön, bei solchen Einstufungen kurz einen Kommentar zu hinterlassen.

Würdet Ihr bitte die GitHub-Issue-Nummer wieder bei den Änderungen in der offiziellen Liste mit angeben. Dies hat eine Weile gut funktioniert, wird aktuell jedoch wieder des Öfteren weggelassen.

Vielen Dank.

Viele Grüße

Daniel

SusanneSchadenberg commented 2 years ago

Hallo Daniel,

zum jetzigen Zeitpunkt kann ich leider noch nicht sagen, wie und wann eine Umsetzung erfolgen wird. Sobald ich Näheres weiß, würde ich mich melden. Was das setzen der GitHub-Issus-Nummer im Änderungsdokument betriff, probieren wir immer die Nummer mit einzutragen, aber manchmal geht es doch unter. Wir werden probieren, darauf wieder mehr zu achten.

Viele Grüße aus Magdeburg Susanne