Added 'ExtraData' to Pr0grammResponse - #24
Conversation
Allowing the Dev to Inspect/Use extra data from the api which could come in future
holzmaster
left a comment
holzmaster
left a comment
There was a problem hiding this comment.
Geht hier auch ein Read-only Dictionary?
|
Ist halt nicht gut, wenn das Feature dann irgendwann drin ist, aber schon Entwickler damit arbeiten und es plötzlich weg ist. |
|
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. |
|
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. |
|
Ja das wär vll. eine bessere Lösung. Ist das denn überhaupt möglich, alle Felder zusätzlich in der dict zu speichern? |
|
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. |
3 participants
Allowing the Dev to Inspect/Use extra data from the api which could come in future