Skip to content

Sorting av Appoptions#42

Merged
ivarne merged 16 commits intoAltinn:mainfrom
StianVestli:Sorting
Nov 6, 2024
Merged

Sorting av Appoptions#42
ivarne merged 16 commits intoAltinn:mainfrom
StianVestli:Sorting

Conversation

@StianVestli
Copy link
Copy Markdown
Contributor

@StianVestli StianVestli commented Oct 28, 2024

Description

Endret GetAppOptionsAsync, lagt til nytt funksjonkall SortAppOptions, samt støtte for nye parametere, orderBy og OrderByDesc.
SortAppOption returnere en sortert liste, hvis det er sent inn paramerter orderBy for stigende (asc) eller orderByDesc (desc) for synkende.
En kan sortere listen på name, code, description, helpText stigende eller synkenden.
Man kan også sortere den etter rekkerfølgen en har angitt selectCodes i hvis en ønsker dette.

Vi har et behov å kunne sortere listene våre, og ofte i bestemte rekkefølger ikke i en sorte rekkefølge.

Verification

  • Your code builds clean without any errors or warnings
  • Manual testing done (required)
  • Relevant automated test added (if you find this hard, leave it and we'll help out)
  • All tests run green

@danielskovli danielskovli requested a review from ivarne November 1, 2024 08:19
Copy link
Copy Markdown
Member

@ivarne ivarne left a comment

Choose a reason for hiding this comment

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

Jeg la til noen tester og hadde noen spørsmål. Kan du ta en titt?

Når jeg ser på dette tenker jeg at orderBy og orderByDesc kanskje burde vært standard funksjonalitet som ikke bare gjelder ssb kokdelsiter, men i såfall gir det vel mest mening å gjøre sorteringen i frontend.

En ting som bare kan gjøres i backend er å sortere på de feltene som ikke er med i Altinn sin forståelse av et option (eg ParentCode, Level, ValidFrom, ValidTo)++. Om det er relevant bør i såfall sorteringen flyttes til før man mapper fra ClassificationCode til AppOption

Comment thread src/Altinn.Codelists/SSB/ClassificationCodelistProvider.cs
Comment thread src/Altinn.Codelists/SSB/ClassificationCodelistProvider.cs
@ivarne ivarne merged commit 211a845 into Altinn:main Nov 6, 2024
@ivarne ivarne added the feature Label Pull requests with new features. Used when generation releasenotes label Nov 6, 2024
@olemartinorg olemartinorg added community-contribution-❤️ Pull request from developers outside the Altinn teams. org/ssb Issues relevant for Statistisk sentralbyrå. labels Nov 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-contribution-❤️ Pull request from developers outside the Altinn teams. feature Label Pull requests with new features. Used when generation releasenotes org/ssb Issues relevant for Statistisk sentralbyrå.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants