refactor: Move Collections-specific code to dedicated package (TT-1475) #53
Conversation
pierrebeauguitte
added
the
refactor
There was a problem hiding this comment.
Ser bra ut, men lurer på om vi også skulle renamet CollectionsService i newspaper-packagen til noe annet, kanskje bare NewspaperService eller noe sånt? I CoreControlleren har vi en oppdeling hvor materialtypen bestemmer katalogtypen, og ut fra det hvilken service som kalles. Ut fra det kan det se ut som CollectionsService er et passende navn, men tror vi må gjøre om på det hvis/når flere materialtyper kommer inn i Collections. Akkurat det siste nok utenfor scopet til denne oppgaven.
Ser det er flere funksjoner og klasser som gjerne skulle ha endret navn, men det kan vi ta i en annen sak etter denne, det vil ikke påvirke andre PRs i noe særlig grad. |
No description provided.