Open brendonn40 opened 3 months ago
The latest updates on your projects. Learn more about Vercel for Git ↗︎
Name | Status | Preview | Comments | Updated (UTC) |
---|---|---|---|---|
enem-api | ❌ Failed (Inspect) | Aug 28, 2024 4:07pm |
@yunger7 Dá uma verificada por gentileza
Muito obrigado pela PR @brendonn40!! Isso vai inclusive resolver alguns problemas de imagens quebradas que estamos tendo atualmente. Tenho alguns pontos pra repassar contigo, mas estou interessado em dar merge nessa PR.
Esses são pontos que precisamos resolver antes do merge.
file
está invalida, com a URL do cloudfront concatenada na URL da API. Por exemplo, na questão 128 do ano 2017 (Segunda aplicação), que possui o valor https://enem.dev/2017-segunda-aplicacao/questions/128/https://d2q576s0wzfxtl.cloudfront.net/2018/04/10150337/128-b.png
. Pelos meus testes, isso parece acontecer apenas na propriedade file
da alternativa, mas bom dar um double-check nas outras propriedades também./exams
. Aqui ainda precisamos dar uma refatorada nos demais endpoints para suportar isso, então por hora, podemos apenas não colocar as provas de segunda aplicação no exams.json
, assim a gente evita o pessoal tentando acessar rotas que não existem.Esses são pontos de baixa prioridade, não são impeditivos para o merge, mas seria bacana fazermos também.
title
, tanto da prova quanto das questões. Aqui, me refiro apenas as novas provas adicionadas (Segunda aplicação). Segue alguns exemplos:
context
da questão. Aqui nesse ponto pode deixar que eu vejo, já tenho uma função pronta, então acredito que o esforço vai ser mais baixo.Fiz o commit ajustando o path das imagens das alternativas, e o titulo das perguntas/ exams. A unica rota atual que tá quebrando é a v1/exams/${year}-segunda-aplicacao, pq ela procura no public/exams.json, e como o campo de year ta como number, se tentar pegar os dados de uma segunda aplicacao, ele não vai achar. Porem a rota /v1/exams/${year}-segunda-aplicacao/questions e v1/exams/{year}-segunda-aplicacao/questions/{index} funcionam normalmente pq ela procura as perguntas pelo diretorio e o param de year ta como string.
@yunger7 Conseguimos seguir com essa PR?
Resumo do Pull Request: Adicionei perguntas dos anos onde aconteceu 2 aplicações do enem no mesmo ano. Mantive o padrão das perguntas. Também adicionei contexto nas perguntas que faltavam.