JSONAPI-Resources / JSONAPI-Resources/jsonapi-resources

Id is required for update of singular resource.

Offen
#976 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
Ruby
Sterne
2.3k
Forks
546
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

Hello. In my case I send put request to /profile route. And since this resource is singular, I don't need to provide id param. But, unfortunately, it throws A key is required error if I don't provide id(it works if I provide any random value as id).

It happens because of this line of code
https://github.com/cerebris/jsonapi-resources/blob/master/lib/jsonapi/request_parser.rb#L201

Are the any specific reasons for this behaviour?

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Lies lib/jsonapi/request_parser.rb ungefähr ab Zeile 201 und verfolge, wie eine PUT-Anfrage an die singuläre Ressource /profile geparst wird. Reproduziere die Fälle mit fehlender ID und beliebiger ID und ermittle anschließend, ob die aktuelle Anforderung eines Schlüssels beabsichtigt ist. Die Aufgabe ist abgeschlossen, wenn das erwartete Verhalten und sein Grund geklärt sind und ein Regressionstest vorhanden ist, falls sich das Verhalten ändern sollte.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
ruby
Bereich
api
Issue-Typ
Bug
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
25/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.