OpenPr0gramm icon indicating copy to clipboard operation
OpenPr0gramm copied to clipboard

Added 'ExtraData' to Pr0grammResponse

Open JKamsker opened this issue 4 years ago • 5 comments

Allowing the Dev to Inspect/Use extra data from the api which could come in future

JKamsker avatar Feb 03 '20 13:02 JKamsker

Ist halt nicht gut, wenn das Feature dann irgendwann drin ist, aber schon Entwickler damit arbeiten und es plötzlich weg ist.

holzmaster avatar Feb 03 '20 17:02 holzmaster

Dafür gibts ja SemVer. Wenn man die upgraded, muss man damit rechnen, dass sich was ändert.

In meinem Fall hat die errormsg beim login gefehlt, dass das chaptcha fehlt.

JKamsker avatar Feb 03 '20 18:02 JKamsker

Ist halt ein ziemlich ekliger breaking change. Ist zwar breaking, aber auf das migrieren wird einen kein compilerfehler hinweisen. Und auch nicht ob das Feld jemals wieder so befüllt sein kann.

Ich wäre vielmehr dafür, ein Dictionary mit allen Feldern anzugeben. Mit sowas kann die Response erweitert werden, aber der bestehende Code wird nicht in einen fragwürdigen Zustand gesetzt.

holzmaster avatar Feb 04 '20 07:02 holzmaster

Ja das wär vll. eine bessere Lösung. Ist das denn überhaupt möglich, alle Felder zusätzlich in der dict zu speichern?

JKamsker avatar Feb 04 '20 07:02 JKamsker

Keine Ahnung. Kann mir vorstellen, dass das mit Newtonsoft JSON irgendwie geht.

Auf lange Sicht wollte ich eigentlich davon weg und eher zu dem JSON-FW des .NET-Standards.

holzmaster avatar Feb 04 '20 08:02 holzmaster