-
Notifications
You must be signed in to change notification settings - Fork 33
Chore/update ds #3231
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
base: main
Are you sure you want to change the base?
Chore/update ds #3231
Conversation
/publish |
PR release:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Phenomenal stuff! 🚀 🌔 A couple of nitpicks from me, but this is a solid piece of work -- good job! 👏 👏 👏
onChange, | ||
...rest | ||
}: PaginationComponentProps) { | ||
const { pages, showNextPage, showPreviousPage } = usePagination({ | ||
compact, | ||
const { pages, prevButtonProps, nextButtonProps, hasPrev, hasNext } = usePagination({ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Github is complaining that compact
is defined but never used in this component now. 🤔
test: /\.tsx?$/, | ||
loader: 'esbuild-loader', | ||
options: { | ||
target: 'es2020', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ooh, this is a good point - yet another reason we might want extensive testing for this one. According to this comment, ES2020 (aka ES11) is more than fine, but this might of course break older stuff.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Forgot the link, but here it is: https://digdir.slack.com/archives/C07A822E9PT/p1749541859505669?thread_ts=1747917879.904519&cid=C07A822E9PT
We should also take a look at all the issues linked to #2600 that we might want to close as solved after this. 🤔 I saw at least one thing I remember being and issue which you seem to have fixed here (I didn't test, just read the code). |
|
Description
Related Issue(s)
Verification/QA
Manual functionality testing
Automated tests
UU/WCAG (follow these guidelines until we have our own)
User documentation @ altinn-studio-docs
Support in Altinn Studio
Sprint board
Labels
kind/*
andbackport*
label to this PR for proper release notes grouping