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.

Sign up for GitHub

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

refactor(pages-api): change parameter for layout operations to just use layout name #14683

Open

Jondyr wants to merge 3 commits into main from spr/main/cf35ff23

Open

refactor(pages-api): change parameter for layout operations to just use layout name #14683

Merge branch 'main' into spr/main/cf35ff23

GitHub Advanced Security / CodeQL succeeded Feb 27, 2025 in 3s

2 new alerts

New alerts in code changed by this pull request

2 notes

See annotations below for details.

View all branch alerts.

Annotations

Check notice on line 55 in backend/src/Designer/EventHandlers/ComponentIdChanged/ComponentIdChangedLayoutsHandler.cs

Code scanning / CodeQL

Missed opportunity to use Select Note

This foreach loop immediately

maps its iteration variable to another variable

- consider mapping the sequence explicitly using '.Select(...)'.

Check notice on line 378 in backend/src/Designer/Infrastructure/GitRepository/AltinnAppGitRepository.cs

Code scanning / CodeQL

Missed opportunity to use Select Note

This foreach loop immediately

maps its iteration variable to another variable

- consider mapping the sequence explicitly using '.Select(...)'.