QAC-Portal/QA-Portal

Slightly Refactor Skills - CV api

Aperta

#183 aperta il 18 dic 2019

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)Java (5 fork)auto 404
Low PriorityUIgood first issuejava

Metriche repository

Star
 (2 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Currently in the CV api, under CvVersion.java, a person's skills are initialised as type List<Skills>.

However, Skills itself is an object made up of a list of skills. It's my understanding that everything can actually be contained within one Skills object, rather than a list of them (a list of lists).

This issue would involve:

  • changing the allSkills variable in CvVersion to be of type Skills rather than List<Skills>

  • making sure nothing breaks in the process

  • rewriting the few CV api tests that refer to the allSkills variable

Guida contributor

Direzione di ricerca
Individuare CvVersion.java nell'API CV, cambiare il tipo di allSkills da List<Skills> a Skills, aggiornare tutti i file di test che fanno riferimento a allSkills ed eseguire la suite di test per verificare che nulla si rompa.
Tech stack
java
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85