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: reimplement report graphs #2149

Merged
merged 3 commits into from
Nov 1, 2024
Merged

Conversation

NilsOveTen
Copy link
Contributor

resolve #2147

@NilsOveTen
Copy link
Contributor Author

Må vente med merge av denne til prodsetting av statistics-service er utført: Informasjonsforvaltning/fdk-statistics-service#3

@NilsOveTen NilsOveTen marked this pull request as draft October 9, 2024 07:45
@NilsOveTen
Copy link
Contributor Author

kommentarer fra kjersti:
bedre om grafene starter på 0, nå ser det veldig dramatisk ut for små endringer
spinner når filterert endres, urelatert graf vises i noen sekunder

@NilsOveTen
Copy link
Contributor Author

kommentarer fra kjersti: bedre om grafene starter på 0, nå ser det veldig dramatisk ut for små endringer spinner når filterert endres, urelatert graf vises i noen sekunder

yaksen starter nå på 0: 474617e
fjerner enkelt og greit grafene hvis org-filteret er aktivt: 42d504d

Tror det krever en del refaktorering om jeg skal få til en spinner på grafene, kan inkludere grafene for filteret når vi tar disse sidene inn i fdk-frontend.

@NilsOveTen NilsOveTen marked this pull request as ready for review October 9, 2024 08:29
@NilsOveTen NilsOveTen force-pushed the feat/not/reimplement-report-graphs branch 2 times, most recently from 41154df to 8704950 Compare October 28, 2024 07:58
@NilsOveTen NilsOveTen force-pushed the feat/not/reimplement-report-graphs branch from 8704950 to 4ba6646 Compare November 1, 2024 06:48
@NilsOveTen NilsOveTen merged commit 19476df into main Nov 1, 2024
2 checks passed
@NilsOveTen NilsOveTen deleted the feat/not/reimplement-report-graphs branch November 1, 2024 06:55
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.

Reimplementer tidsserie-visning på rapportsidene
3 participants