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

feat: add new csv view with total for gangs #1635

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

magsyg
Copy link
Contributor

@magsyg magsyg commented Dec 8, 2024

closes #1432

Also added the other csv view for recruitment as a whole

@magsyg magsyg self-assigned this Dec 8, 2024
Copy link
Contributor

@Snorre98 Snorre98 left a comment

Choose a reason for hiding this comment

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

Denne løser ikke det andre problemet nevt i issuet. I CSVen som lages for en gang skal kun antall intervjuer for den gjengen vises. I tillegg innser jeg at det kanskje er best å gjøre det samme for søkers prioritet. Så da burde kanskje tittelen på kolonnen bli "Søkers rangering av verv i denne gjengen" og "Intervjuer satt i denne gjengen"
I dette bilde har f.eks. Simen kun søkt to stillinger i VK, men det vises x av 4 intervjuer satt.
bilde

@@ -135,6 +135,11 @@
views.RecruitmentRecruiterDashboardView.as_view(),
name='recruitment_recruiter_dashboard',
),
path(
'recruitment-download-applications-csv/<int:recruitment_id>/',
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
'recruitment-download-applications-csv/<int:recruitment_id>/',
'recruitment-download-all-applications-csv/<int:recruitment_id>/',

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@@ -1150,6 +1150,63 @@ def get(
return Response(data=RecruitmentPositionSharedInterviewGroupSerializer(interview_groups, many=True).data, status=status.HTTP_200_OK)


class DownloadRecruitmentApplicationCSV(APIView):
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
class DownloadRecruitmentApplicationCSV(APIView):
class DownloadAllRecruitmentApplicationCSV(APIView):

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@magsyg
Copy link
Contributor Author

magsyg commented Dec 11, 2024

Denne løser ikke det andre problemet nevt i issuet. I CSVen som lages for en gang skal kun antall intervjuer for den gjengen vises. I tillegg innser jeg at det kanskje er best å gjøre det samme for søkers prioritet. Så da burde kanskje tittelen på kolonnen bli "Søkers rangering av verv i denne gjengen" og "Intervjuer satt i denne gjengen" I dette bilde har f.eks. Simen kun søkt to stillinger i VK, men det vises x av 4 intervjuer satt. bilde

Oja lol plasserte for gang i feil endepunkt

@magsyg
Copy link
Contributor Author

magsyg commented Dec 11, 2024

Denne løser ikke det andre problemet nevt i issuet. I CSVen som lages for en gang skal kun antall intervjuer for den gjengen vises. I tillegg innser jeg at det kanskje er best å gjøre det samme for søkers prioritet. Så da burde kanskje tittelen på kolonnen bli "Søkers rangering av verv i denne gjengen" og "Intervjuer satt i denne gjengen" I dette bilde har f.eks. Simen kun søkt to stillinger i VK, men det vises x av 4 intervjuer satt. bilde

Rangering i gjengen føler jeg ikke er nødvendig? Er vell uansett viktigere for folk å ha rangeringen til posisjonen? feks man er 5. plass på brukers rangering, så blir det misvisende å gi den rangering 1 dersom det er eneste den har søkt i gjengen

@Snorre98
Copy link
Contributor

Snorre98 commented Dec 20, 2024

Denne løser ikke det andre problemet nevt i issuet. I CSVen som lages for en gang skal kun antall intervjuer for den gjengen vises. I tillegg innser jeg at det kanskje er best å gjøre det samme for søkers prioritet. Så da burde kanskje tittelen på kolonnen bli "Søkers rangering av verv i denne gjengen" og "Intervjuer satt i denne gjengen" I dette bilde har f.eks. Simen kun søkt to stillinger i VK, men det vises x av 4 intervjuer satt. bilde

Rangering i gjengen føler jeg ikke er nødvendig? Er vell uansett viktigere for folk å ha rangeringen til posisjonen? feks man er 5. plass på brukers rangering, så blir det misvisende å gi den rangering 1 dersom det er eneste den har søkt i gjengen

Ja, jeg kan se det vil være nyttig å ha oversikten som den er nå for rangering. Jeg tenke på at det kanskje ikke burde være mulig å se antall søkander gjennom CSVen, men gitt at kun opptaksansvarlig i gjengen skal ha tilgang til funksjonene så er det kanskje greit.

Men da vil jeg foreslå at kolonnen tittel ( celle J1) spesifisere at det er snakk om datapunkter for hele opptaket. F.eks. kan J1 være "Søkers rangering (hele opptaket)".

Jeg tror det er mer oversiktlig/mer nyttig hvis kolonne K viser til data fra gjengens opptak. Det gjør det enklere for opptaksanvarlig i gjengene å se om det mangler intervjuer i gjengen.

Så, løsning: La kolonne J være, men endre tittel + endre data i kolonne K til å kun vise til gjengens data.

@magsyg
Copy link
Contributor Author

magsyg commented Dec 20, 2024

Det er fikset

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.

Fix exported CSV
2 participants