Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Comments

Close side panel

[OGUI-1620] Create services with business logic for layouts#3115

Draft
mariscalromeroalejandro wants to merge 15 commits intodevAliceO2Group/WebUi:devfrom
feature/QCG/OGUI-1620/layout-management-servicesAliceO2Group/WebUi:feature/QCG/OGUI-1620/layout-management-servicesCopy head branch name to clipboard
Draft

[OGUI-1620] Create services with business logic for layouts#3115
mariscalromeroalejandro wants to merge 15 commits intodevAliceO2Group/WebUi:devfrom
feature/QCG/OGUI-1620/layout-management-servicesAliceO2Group/WebUi:feature/QCG/OGUI-1620/layout-management-servicesCopy head branch name to clipboard

Conversation

@mariscalromeroalejandro
Copy link
Contributor

@mariscalromeroalejandro mariscalromeroalejandro commented Oct 10, 2025 •
edited
Loading

I have JIRA issue created

  • branch and/or PR name(s) includes JIRA ID
  • issue has "Fix version" assigned
  • issue "Status" is set to "In review"
  • PR labels are selected
  • FLP integration tests were ran successful

This PR introduces dedicated services for handling layout data. All business logic is applied within the service layer before data reaches the repositories.

  • Created helpers that keep charts, grids, and tabs in sync.
  • New service layer that centralizes layout-related business logic before data reaches repositories.
  • User service to support user-related operations that interact with layout data.

github-advanced-security[bot]
github-advanced-security bot found potential problems Oct 10, 2025
View reviewed changes
QualityControl/test/lib/services/layout/helpers/layoutMapper.test.js Fixed Show fixed Hide fixed
QualityControl/test/lib/services/layout/helpers/layoutMapper.test.js Fixed Show fixed Hide fixed
QualityControl/test/lib/services/layout/helpers/layoutMapper.test.js Fixed Show fixed Hide fixed
QualityControl/test/lib/services/layout/helpers/layoutMapper.test.js Fixed Show fixed Hide fixed
QualityControl/test/lib/services/layout/helpers/layoutMapper.test.js Fixed Show fixed Hide fixed
@mariscalromeroalejandro @github-advanced-security
Potential fix for code scanning alert no. 260: Superfluous trailing a…
a41f05f 
…rguments

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@mariscalromeroalejandro mariscalromeroalejandro mentioned this pull request Oct 24, 2025
Draft
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@graduta graduta Awaiting requested review from graduta graduta will be requested when the pull request is marked ready for review graduta is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@mariscalromeroalejandro mariscalromeroalejandro

Labels

Backend Quality Control

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@mariscalromeroalejandro
Morty Proxy This is a proxified and sanitized view of the page, visit original site.