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

Rename GangType to Department #1506

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Conversation

Frenje123
Copy link
Member

GangType is now called Department.

Fixes #1278

@Frenje123 Frenje123 added the refactor Clean up code without changing functionality label Oct 1, 2024
@Frenje123 Frenje123 linked an issue Oct 1, 2024 that may be closed by this pull request
@robines
Copy link
Member

robines commented Oct 1, 2024

Helt imot dette, GangType gir fullstendig mening. Drift, kunstneriske, arragerende osv er ikke departementer, de er bare en kategorisering av gjengene!! Kan da heller rename til GangCategory, hvis folk faktisk ikke skjønner GangType

Copy link
Contributor

@Mathias-a Mathias-a left a comment

Choose a reason for hiding this comment

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

Enig med Robin

Copy link
Contributor

Choose a reason for hiding this comment

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

Starter en thread jeg. Enig med Robin. Departments er et helt fremmed ord i Samfundet kontekst.

Jeg tror at hvis man hadde lest "department" hadde man aldri ever skjønt at det refererte til "kunstneriske", "Drift". Enig i at det ikke er intuitivt med GangType heller, men da er det iaf 5% sjanse for at noen klarer å gjette det?

Copy link
Contributor

Choose a reason for hiding this comment

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

Jeg liker department. Syns GangType var ekstremt forvirrende første gang jeg leste det. Syns ikke det burde ha noe å si om det passer inn i vokabularet på Samfundet. Bruker vi ikke "Sulten" nettopp fordi vi prøver å unngå å referer til Samfundet vokabular?

Og i tillegg: nå har jo Marcus brukt litt tid på denne PR-en, så vi må finne en måte å bli enig om slike ting før arbeidet blir gjort. Ganske sikker på at Robin i det minste har vært kjent med dette issuet. Det burde blitt diskutert mye tidligere, spesielt hvis man har spesifikte ønsker som "GangCategory".

Copy link
Member

Choose a reason for hiding this comment

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

Syns GangType var ekstremt forvirrende første gang jeg leste det

Jeg kan egentlig si meg enig i at det ikke er intuitivt nøyaktig hva det betyr, men etter man får det forklart én gang så sitter det jo. Det er mange andre navn og konsepter i Samfundet4 som er mye mer forvirrende enn akkurat dette syns jeg, selv etter man har fått det forklart flere ganger (feks application priority vs. status).

Departement er hundre ganger verre. Det bringer inn alt for mange assosiasjoner, og ordet refererer nesten alltid til en organisatorisk struktur/enhet. Og det er jo litt dumt, siden det ikke finnes noen gjeng-departementer på Samf! Derfor ville det ha vært utrolig forvirrende å introdusere det som et begrep i prosjektet. Man har ikke det samme problemet med “type” eller “category” fordi de er såpass vage begrep, og man skjønner intuitivt at da handler det om “hva slags type gjeng er det”, i motsetning til “hvilket departement tilhører denne gjengen”.

Syns ikke det burde ha noe å si om det passer inn i vokabularet på Samfundet. Bruker vi ikke "Sulten" nettopp fordi vi prøver å unngå å referer til Samfundet vokabular?

Apples and oranges. Sulten heter Sulten fordi Lyche endret navn way back in the day, og da tenkte man at man vil unngå refaktorering om det skulle skje igjen. Så problemet er ikke “om det passer inn i vokabularet på Samfundet”, men heller at her virker det som om man prøver å introdusere et ord som om det faktisk var i vokabularet.

Og i tillegg: nå har jo Marcus brukt litt tid på denne PR-en, så vi må finne en måte å bli enig om slike ting før arbeidet blir gjort.

Det er synd ja, men jeg var ikke klar over at dette foregikk, som sikkert skyldes at Marcus ikke har vært på innsjekksrunden på en stund. Hadde noen sagt at de jobbet med dette på runden hadde jeg protestert.

Ganske sikker på at Robin i det minste har vært kjent med dette issuet.

Jeg husker at du har tatt opp dette som en mulighet tidligere to-tre ganger. Kanskje jeg ikke har vært tydelig nok, men jeg mener at jeg har svart at jeg var imot det hver gang.

Angående GangType vs GangCategory syns jeg GangCategory er mest intuitiv, men forskjellen på de syns jeg er for liten til å være verdt å bruke tid på å refaktorere. Men nå er vi jo her, så da kan vi like godt endre til GangCategory hvis alle er enige?

Copy link
Member

@emilte emilte Oct 3, 2024

Choose a reason for hiding this comment

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

Navnebytte på slike variabler er som oftest bare noen få minutters jobb, så tipper at lite tid er sløst. Dere ville oppdaget dette tidligere også hvis dere deler opp i mindre deler i stedet for slike kembo PR som jeg har anbefalt tidligere.

Copy link
Member

Choose a reason for hiding this comment

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

Dere bør kanskje tagge issues som needs-discussion

Copy link
Contributor

Choose a reason for hiding this comment

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

@Frenje123 (etter diskusjon på arbeidskveld) Hva tenker du om å bytte til GangCategory?

@Snorre98
Copy link
Contributor

Snorre98 commented Oct 17, 2024

Jeg endrer denne til draft. Lukker ikke i tilfelle det ødlegger for de som diskuterer

@Snorre98 Snorre98 marked this pull request as draft October 17, 2024 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor Clean up code without changing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Rename gangType
5 participants