code-corps / code-corps/code-corps-ember

Modify Conversation serializer to handle includes based on JSON API sideposting draft

Offen
#1,624 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
blocked
Vorherrschende Sprache
JavaScript
Sterne
120
Forks
75
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

# Problem

From https://github.com/json-api/json-api/pull/1197/files?short_path=571f6ce#diff-571f6cee9a492e0e177801c21a4b2b2a

When creating a `conversation` alongside a `message`, we need to generate a `tempId` for that conversation. The `tempId` needs to appear as `temp-id` both in the json['data']['relationship'] identifier as well as the `json['included']` section and needs to be posted to the API.

## References

Needs to be implemented together with #1303 https://github.com/code-corps/code-corps-api/issues/1303.

The code this needs to modify is in #1597, so this should be considered blocked until that is merged.

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginne mit der Prüfung der Serializer-Änderungen aus #1597, die dieses Issue als Voraussetzung identifiziert, und lies #1303 für die zugehörige Implementierung. Überprüfe den Sideposting-Entwurf der JSON API und verfolge die Erstellung einer Konversation zusammen mit einer Nachricht. Erledigt bedeutet, dass die generierte tempId sowohl im Beziehungsbezeichner als auch in json['included'] als temp-id ausgegeben und anschließend an die API gesendet wird.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
javascript
Bereich
frontend, web-dev
Issue-Typ
Feature
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
25/100

Neue Issues direkt in Ihr Postfach

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