search

diogolundberg / educat-erp

Educat-erp ⚡️
0 stars 0 forks source link

CRUD do onboarding #112

Closed diogolundberg closed 6 years ago

shff commented 6 years ago

@lucascostadev O GET da API não está retornando o ID, o que impossibilita o uso do PUT e do DELETE

{
    "records": [
        {
            "semester": "1",
            "year": "2018",
            "startAt": "01/06/2018",
            "endAt": "31/12/2018",
            "enrollmentCount": "8"
        }
    ]
}
lucascostadev commented 6 years ago

@shferreira. Beleza vou olhar. Vc acha que quando for assim (não der erro 500). Para não atrapalhar o fluxo, vc possa pegar do banco o id e fazer os testes nos outros endpoints.

É algo que corrijo bem rápido. Mas como trabalhamos de forma assíncrona. Acaba que perdemos um tempo considerável.

O que acha. ?

lucascostadev commented 6 years ago

Acabei de publicar a correção. Quando puder só testar.

shff commented 6 years ago

@lucascostadev Opa, eu já tô pegando do banco mesmo, o desenvolvimento em si não fica travado! A questão é mais de não poder subir a mudança pra master mesmo!

shff commented 6 years ago

@lucascostadev @diogolundberg Ok, agora tá faltando o endpoint de GET...

Sem ele não dá pra dar PUT (senão eu perco dados), ou pra listar os alunos

captura de tela 2018-04-30 as 21 53 22

lucascostadev commented 6 years ago

@shferreira publiquei uma versão aqui com ele.

shff commented 6 years ago

@lucascostadev Opa, funcionou 100%

shff commented 6 years ago

Ok, acredito que agora está tudo certo