Feature/species groups statistics #1373
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
AneMarlene
requested changes
Jan 2, 2024
There was a problem hiding this comment.
Tjohei - det blir bra med en "merk alle" på Rødlistefilteret også👍 👍
Jeg satte på meg granske-brillene og fant sannelig noe her og der som må fikses før den kan gå på lufta. Jeg har for øvrig ikke sett på "Forberedelse til #1369", ettersom jeg regner med reviewen på det kommer når figuren er på plass(?!) :)
I tillegg til de øvrige kommentarene som er filter-spesifikke, så ser jeg på et overordnet nivå at filtertelleren har blitt feil. Den teller nå med "merk alle" som et (reelt) filter. Jeg tror du har en fiks på det inne i fremmedartskoden et sted (da vi hadde samme issue der en gang)?
...nts.Frontend.Web/Infrastructure/RedlistSpecies/Enums/RedlistSpeciesAssessment2021Category.cs
Show resolved
Hide resolved
Assessments.Frontend.Web/Infrastructure/RedlistSpecies/FilterItemHelpers.cs
Outdated
Show resolved
Hide resolved
...nts.Frontend.Web/Infrastructure/RedlistSpecies/Enums/RedlistSpeciesAssessment2021Habitats.cs
Outdated
Show resolved
Hide resolved
|
Da er feilene over utbedret. |
AneMarlene
approved these changes
Jan 17, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Gjør om virkemåten for filter på rødlista for arter til å fungere likt som for fremmedartslista.
Fixes #264 for filtre med flere enn 4 valg, slik som for fremmedartslista.
Forberedelse til #1369
Her har jeg testa at filter fungerer likt som det gjør i produksjon i dag, men det er fint om flere enn meg tester dette.