Skip to content
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

En el curs 20-21 les assignatures de Q2 són any 2020 o 2021? #76

Open
PauMatas opened this issue Mar 18, 2024 · 4 comments
Open

En el curs 20-21 les assignatures de Q2 són any 2020 o 2021? #76

PauMatas opened this issue Mar 18, 2024 · 4 comments
Labels
question Further information is requested

Comments

@PauMatas
Copy link
Member

https://github.com/data-students/apunts-dades/blame/dca480cb5720c820e8a43b638445ab478a1bf7b5/src/components/Form.tsx#L193

Se que hauria d'haver fet això a la teva PR (@marcfranquesa ) i se que pot sonar molt de rata i bla bla bla... Però, no hauria de ser generacio + Math.floor(semester / 2) en comptes de generacio + Math.floor((semester - 1) / 2). D'aquesta manera les assignatures de Q parells (semestre de primavera) ja tindrien l'any de la segona part del curs, m'explico?

@PauMatas PauMatas added the question Further information is requested label Mar 18, 2024
@marcfranquesa
Copy link
Member

marcfranquesa commented Mar 18, 2024

Pot ser sisi, jo només vaig copiar la fórmula que tenieu, jo de fet seria fan de ficar 2021/2022 que trobo que seria més fàcil d'entendre, crec que seria només canviar el label? que crec que és fàcil i no cal canviar-ho a la BD

@PauMatas
Copy link
Member Author

Jo ho veig bé! @puigde ?

@puigde
Copy link
Contributor

puigde commented Mar 18, 2024

sounds good anyinici/anyfinal s'enten millor sí

@puigde
Copy link
Contributor

puigde commented Mar 18, 2024

només que fa una mica de mandra perquè s'han de re-fer els valors que tenim a producció i és com una cosa poc prio sota la meva opinió

@PauMatas PauMatas linked a pull request Mar 18, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants