-
Notifications
You must be signed in to change notification settings - Fork 48
feat: added questions and fixed the ones without context #7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
@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. BlockingEsses são pontos que precisamos resolver antes do merge.
Non-blockingEsses são pontos de baixa prioridade, não são impeditivos para o merge, mas seria bacana fazermos também.
|
|
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? |
|
@yunger7 tudo certo com esse PR? |
Acredito que ele não quer mais seguir com esse projeto. |
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.