Skip to content

Conversation

@muratmerdoglu-dp
Copy link
Contributor

Short Description

Links to Ticket and related Pull-Requests

Changes

Data-security

Deployment

New Repos, NPM packages or vendor scripts

Screenshots of UI changes

Checklist before merging

  • QA: In addition to review, the code has been manually tested (if manual testing is possible)
  • PO: Any deviation from requirements was agreed with Product-Owner / ticket author / support-team
  • DEV: Every new component is implemented having accessibility in mind (e.g. aria-label, role property)
  • Cypress: Every new feature has suitable Cypress tests implemented

Notice: Please keep this Pull-Request as a Draft (or add WIP label), until it is ready to be reviewed

@sebmue-dataport
Copy link
Contributor

I like the way it is now with the individual functions so it is easier to understand. I think we just need another test to make sonarqube happy and can get it merged

@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
54.2% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@MartinSchuhmacher
Copy link
Contributor

Topic will be done in BC-10814 additionally to moving the logic from FE to BE and only let the FE handle as much as necessary (moving the whole logic from FE to BE might be done in a later ticket if it gets too big).

To not put any more logic to the FE we will not continue with this ticket.

-> closed

@MartinSchuhmacher MartinSchuhmacher deleted the BC-10803-bug-fix-admin branch October 24, 2025 07:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants