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

Feature/statsfrontend1 #1251

Merged
merged 6 commits into from
Sep 12, 2024
Merged

Feature/statsfrontend1 #1251

merged 6 commits into from
Sep 12, 2024

Conversation

magsyg
Copy link
Contributor

@magsyg magsyg commented Jun 30, 2024

Add dto, api and display of those stats through charts and table to RecruitmentsStatsPage

Encountered some problems, but mainly with styling mostly of the charts and sizes.
bilde
bilde

there are some issues encountred:
#1250, piechart fails to display

@magsyg magsyg self-assigned this Jun 30, 2024
@magsyg
Copy link
Contributor Author

magsyg commented Jun 30, 2024

@magsyg Det er ikke bare fordi charts får inn data på uforventet format?

Skeptisk til å endre stylingen med mindre det er tydeligere hvorfor det trengs.

Ser ut som det viser noe data dersom dersom flere av objektene har verdi. I tilfelle her var det kun et av campus stat objektene som har en verdi som er 1.
Med mer data ser det riktig ut
bilde

@Snorre98
Copy link
Contributor

@magsyg Det er ikke bare fordi charts får inn data på uforventet format?
Skeptisk til å endre stylingen med mindre det er tydeligere hvorfor det trengs.

Ser ut som det viser noe data dersom dersom flere av objektene har verdi. I tilfelle her var det kun et av campus stat objektene som har en verdi som er 1. Med mer data ser det riktig ut bilde

Gjør det egentlig det? Nå ser det jo ut som jeg har gjort en feil hvor proposjonene av "kakestykkene" er feil realtivt til "størrelsen" på data de representerer.

@magsyg
Copy link
Contributor Author

magsyg commented Jun 30, 2024

@magsyg Det er ikke bare fordi charts får inn data på uforventet format?
Skeptisk til å endre stylingen med mindre det er tydeligere hvorfor det trengs.

Ser ut som det viser noe data dersom dersom flere av objektene har verdi. I tilfelle her var det kun et av campus stat objektene som har en verdi som er 1. Med mer data ser det riktig ut bilde

Gjør det egentlig det? Nå ser det jo ut som jeg har gjort en feil hvor proposjonene av "kakestykkene" er feil realtivt til "størrelsen" på data de representerer.

bilde

Er sånn den er oppsatt hvorav dataen er label:string, value: number

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.

lgmt

@Snorre98
Copy link
Contributor

Det er kanske et issue på å fikse kakediagram?

@magsyg magsyg merged commit bdfd411 into master Sep 12, 2024
5 checks passed
@magsyg magsyg deleted the feature/statsfrontend1 branch September 12, 2024 11:02
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.

2 participants