Skip to content
Snippets Groups Projects

CE-2952: Addressing security hotspots and code smells (sonarqube)

Merged CE-2952: Addressing security hotspots and code smells (sonarqube)
Merged Max Frederiksen requested to merge CE-2952 into develop

Refactoring and modifications.

Read more about the errors here: https://sonarqube.esss.lu.se/dashboard?branch=develop&id=eu.ess.ics%3Ace-deploy-ui

This MR is now dependent on this MR from common: ce-ui-common!154 (merged)

Before merge:

  • Deploy to test/demo and make sure nothing is crashing
  • Merge common branch and integrate

Follow-ups:

  • Implement this function wherever we use the React key prop
  • Add typing for modules coming from common in “typings.d.ts” to fix syntax highlighting & more
Edited by Max Frederiksen

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading