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

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

Aberta
#1,624 1 comentário 0 reações 0 responsáveis Ver no GitHub
blocked
Linguagem predominante
JavaScript
Estrelas
120
Forks
75
Métricas de merge de PRs
Nenhum PR com merge em 30d

Descrição

# 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.

Guia de contribuição

Abrir o guia de contribuição

Direção de pesquisa

Comece revisando as alterações do serializer de #1597, que esta issue identifica como um pré-requisito, e leia #1303 para consultar a implementação relacionada. Verifique o rascunho de sideposting da JSON API e acompanhe a criação de uma conversa junto com uma mensagem. Considera-se concluído quando o tempId gerado é emitido como temp-id tanto no identificador do relacionamento quanto em json['included'], e depois enviado à API.

Escrita pelo modelo de indexação a partir do texto da issue.

Avaliação

Stack de tecnologia
javascript
Domínio
frontend, web-dev
Tipo de issue
Funcionalidade
Dificuldade
4/5
Tempo estimado
3-5 dias
Status de atividade
Estagnada
Clareza
Razoavelmente clara
Facilidade para iniciantes
25/100

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.