Skip to content

Conversation

@brendonn40
Copy link

@brendonn40 brendonn40 commented Aug 23, 2024

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.

@vercel
Copy link

vercel bot commented Aug 23, 2024

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

@brendonn40 brendonn40 changed the title feat: added questions feat: added questions and fixed the ones without context Aug 24, 2024
@Suburbanno
Copy link

@yunger7 Dá uma verificada por gentileza

@yunger7 yunger7 added enhancement New feature or request good first issue Good for newcomers labels Aug 26, 2024
@yunger7
Copy link
Owner

yunger7 commented Aug 26, 2024

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.

Blocking

Esses são pontos que precisamos resolver antes do merge.

  • Em alguns casos, a propriedade 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.
  • Da forma atual, o único endpoint em que os dados das segundas aplicações vão estar disponíveis é o /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.

Non-blocking

Esses são pontos de baixa prioridade, não são impeditivos para o merge, mas seria bacana fazermos também.

  • Padronização dos valores da propriedade title, tanto da prova quanto das questões. Aqui, me refiro apenas as novas provas adicionadas (Segunda aplicação). Segue alguns exemplos:
    • Título de prova: atualmente o padrão está "ENEM 2011", e nas novas provas está "Prova 2011-segunda-aplicacao". Bom remover o kebab-case, ex: "Prova 2011 (Segunda aplicação)"
    • Título de questão: atualmente o padrão está "Questão 1 - ENEM 2011", e nas novas provas está "Questão 01 da prova cinza do primeiro dia do Enem 2011 Segunda Aplicação".
  • Usar markdown na propriedade 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.

@brendonn40
Copy link
Author

brendonn40 commented Aug 28, 2024

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.

@Suburbanno
Copy link

@yunger7 Conseguimos seguir com essa PR?

@ioolliver
Copy link

@yunger7 tudo certo com esse PR?

@Suburbanno
Copy link

@yunger7 tudo certo com esse PR?

Acredito que ele não quer mais seguir com esse projeto.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request good first issue Good for newcomers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants