Skip to content

[Feature] Habilitar para a criação de novos currículos e Universidades #4

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

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

ItaloSa
Copy link
Member

@ItaloSa ItaloSa commented Jun 14, 2022

Este PR habilita o APP a da suporte a novos currículos.
Para adicionar um novo, basta seguir o padrão (sugiro criar algum tipo de doc definindo isso) que temos pronto em public/university/ufpe/engenharia-da-computacao.json.
O formato da rota ficou /:university/course (e.g. ufpe/engenharia-da-computacao);

Ps: Depois dessa feature vamos ficar sem a rota principal, somente com a do currículo de engenharia da computação.

OLD #2

Copy link
Member

@SteffanoP SteffanoP left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Acho que é necessário um refactor da estrutura curricular de ciência da computação da UFRPE implementado em #3 para obedecer ao padrão implementado. Dá uma olhada no comentário.

Comment on lines +2 to +13
{
"key": "UFPE",
"name": "Universidade Federal de Pernambuco",
"website": "",
"logo": "",
"courses": [
{
"key": "engenharia-da-computacao",
"name": "Engenharia da Computação"
}
]
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

É necessário refatorar e adicionar o currículo de Ciência da Computação da UFRPE, que está em ./public/university/UFRPE/cienciaDaComputacao.json para o padrão utilizado nesse documento.

@@ -52,6 +53,7 @@
]
},
"devDependencies": {
"@types/react-router-dom": "^5.3.1",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Por algum motivo o yarn tá me mostrando um aviso de dependência (abaixo), pode ser um falso negativo, mas dá uma olhada se tá tudo certinho.

warning " > [email protected]" has incorrect peer dependency "eslint@^3.0.0 || ^4.0.0 || ^5.0.0 || ^6.0.0".

@JDaniielC
Copy link

Turma, colocamos novas rotas no projeto adicionando mais cursos da UFPE, através de um script que varre o PDF do perfil curricular:
https://github.com/Abagatinho/visual-curriculum
Seria interessante juntar as duas soluções, utilizando o router do nosso projeto e o sistema de roteamento que vocês estão propondo.

@SteffanoP

This comment was marked as resolved.

@SteffanoP SteffanoP self-assigned this Jun 23, 2022
@JDaniloC

This comment was marked as resolved.

@SteffanoP

This comment was marked as resolved.

@ItaloSa

This comment was marked as resolved.

@JDaniloC
Copy link

Vou ver se consigo sair com alguma solução que junte os códigos

Fala brother, eu acabei de fazer isso no PR19, infelizmente eu não resisti e acabei deixando tudo praticamente em um commit.
O que eu não fiz foi utilizar um service que interliga as funções auxiliares, mas espero que com o estado atual lá do nosso fork fique mais fácil de vocês seguirem a ideia de criação de novos currículos ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants