Skip to content

Feat/16446 navigation validation#4016

Draft
Magnusrm wants to merge 14 commits intomainfrom
feat/16446-navigation-validation
Draft

Feat/16446 navigation validation#4016
Magnusrm wants to merge 14 commits intomainfrom
feat/16446-navigation-validation

Conversation

@Magnusrm
Copy link
Contributor

@Magnusrm Magnusrm commented Feb 25, 2026 •
edited
Loading

Description

Related Issue(s)

Verification/QA

  • Manual functionality testing
    • I have tested these changes manually
    • Creator of the original issue (or service owner) has been contacted for manual testing (or will be contacted when released in alpha)
    • No testing done/necessary
  • Automated tests
    • Unit test(s) have been added/updated
    • Cypress E2E test(s) have been added/updated
    • No automatic tests are needed here (no functional changes/additions)
    • I want someone to help me make some tests
  • UU/WCAG (follow these guidelines until we have our own)
    • I have tested with a screen reader/keyboard navigation/automated wcag validator
    • No testing done/necessary (no DOM/visual changes)
    • I want someone to help me perform accessibility testing
  • User documentation @ altinn-studio-docs
    • Has been added/updated
    • No functionality has been changed/added, so no documentation is needed
    • I will do that later/have created an issue
  • Support in Altinn Studio
    • Issue(s) created for support in Studio
    • This change/feature does not require any changes to Altinn Studio
  • Sprint board
    • The original issue (or this PR itself) has been added to the Team Apps project and to the current sprint board
    • I don't have permissions to do that, please help me out
  • Labels
    • I have added a kind/* and backport* label to this PR for proper release notes grouping
    • I don't have permissions to add labels, please help me out

walldenfilippa and others added 9 commits January 30, 2026 09:39
@walldenfilippa
#17511 + #17512 (#3977)
e985e0e 
Co-authored-by: walldenfilippa <filippa.walden@digdir.no>
Override validation on navigationbuttons
4da9336
minor clean up
acf1099
added override validation on the navigationBar component
8d44e6e
added ovverride customButton + configured the validateOnNext/Forward …
78d28f6 
…and validateOnPrevious/Backward with preventNavigation prop for validationOnNavigation
@Magnusrm
Feat/prevent-navigation (#3990)
7afb86b 
* add basic logic for preventing navigation

* change wording
@walldenfilippa
added PageValidation on layoutSets and layoutSettings (#3998)
2381b63 
* added PageValidation on layoutSets and layoutSettings

---------

Co-authored-by: walldenfilippa <filippa.walden@digdir.no>
@walldenfilippa @Magnusrm
Feat/refactor pagevalidation navigation (#4005)
b0cd6c2 
* refactor: navigation validation backward and improve page validation hooks

* refactor preventing navigation to use simplified config

---------

Co-authored-by: walldenfilippa <filippa.walden@digdir.no>
Co-authored-by: Magnus Revheim Martinsen <mrmartinsen.96@gmail.com>
@walldenfilippa
added prop expandedByDefault to groups for sidenavigation (#4006)
4aad7bf 
* added prop expandedByDefault to groups for sidenavigation

* added prop expandedByDefault to subform sidenavigation

---------

Co-authored-by: walldenfilippa <filippa.walden@digdir.no>
@Magnusrm Magnusrm linked an issue Feb 25, 2026 that may be closed by this pull request
Navigasjonsmenyen: vise instanser i stedet for antall under Subform #3213
Closed
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 25, 2026 •
edited
Loading

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 32967806-d666-4ce1-a9df-553e8db8f778

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/16446-navigation-validation

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@Magnusrm Magnusrm removed a link to an issue Feb 25, 2026
Navigasjonsmenyen: vise instanser i stedet for antall under Subform #3213
Closed
@Magnusrm Magnusrm added backport-ignore This PR is a new feature and should not be cherry-picked onto release branches kind/product-feature Pull requests containing new features labels Feb 25, 2026
Magnusrm and others added 5 commits February 25, 2026 09:39
@Magnusrm
Merge remote-tracking branch 'origin/main' into feat/16446-navigation…
4974eb4 
…-validation
Updated the validation hierarchy for the navigation rules.
78728ee
@Magnusrm
fix prevent navigation
35f730f
@walldenfilippa
feat: enhance navigation validation by integrating validation checks (#…
2fa1241 
…4027)

* feat: enhance navigation validation by integrating validation checks in navigation components

* removed one hook and added useGetNavigationIsPrevented on pageGroup aswell

* temporary notes

* remove temporary note

* update tests

---------

Co-authored-by: walldenfilippa <filippa.walden@digdir.no>
merge main into feat/16446-navigation-validation
4ddf784
@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 10, 2026

Quality Gate Passed Quality Gate passed

Issues
7 New issues
0 Accepted issues

Measures
0 Security Hotspots
72.2% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@walldenfilippa
Copy link
Contributor

walldenfilippa commented Mar 10, 2026

/publish

@github-actions
Copy link
Contributor

github-actions bot commented Mar 10, 2026 •
edited
Loading

PR release:

  • <link rel="stylesheet" type="text/css" href="https://altinncdn.no/toolkits/altinn-app-frontend/4.25.3-pr.4143.16446-navigation-validation.4ddf7840/altinn-app-frontend.css">
  • <script src="https://altinncdn.no/toolkits/altinn-app-frontend/4.25.3-pr.4143.16446-navigation-validation.4ddf7840/altinn-app-frontend.js"></script>

⚙️ Building...
✅ Done!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

backport-ignore This PR is a new feature and should not be cherry-picked onto release branches kind/product-feature Pull requests containing new features

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Forhindre fri navigasjon med sidemeny-navigasjon

2 participants

@Magnusrm @walldenfilippa