Skip to content

fix(menu): highlighted network security dashboard menu#10927

Merged
JacquesLarique merged 1 commit intomasterfrom
UXCT-527-network-security-dashboard-menu-bolder
Feb 29, 2024
Merged

fix(menu): highlighted network security dashboard menu#10927
JacquesLarique merged 1 commit intomasterfrom
UXCT-527-network-security-dashboard-menu-bolder

Conversation

@Steffy29
Copy link
Contributor

Question Answer
Branch? master
Bug fix? yes
New feature? no
Breaking change? no
Tickets Fix #UXCT-527
License BSD 3-Clause
  • Try to keep pull requests small so they can be easily reviewed.
  • Commits are signed-off
  • Only FR translations have been updated
  • Branch is up-to-date with target branch
  • Lint has passed locally
  • Standalone app was ran and tested locally
  • Ticket reference is mentioned in linked commits (internal only)
  • Breaking change is mentioned in relevant commits

Description

Highlighted network security dashboard menu when selected

Related

ref: UXCT-527

Signed-off-by: Stephanie Moallic <stephanie.moallic@corp.ovh.com>
@Steffy29 Steffy29 self-assigned this Feb 15, 2024
@Steffy29 Steffy29 requested a review from a team February 15, 2024 10:54
@Steffy29 Steffy29 requested a review from a team as a code owner February 15, 2024 10:54
@Steffy29 Steffy29 added the bug Something isn't working label Feb 15, 2024
@sonarqubecloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues

Measures
0 Security Hotspots
No data about Coverage
100.0% Duplication on New Code

See analysis details on SonarCloud

icon: getIcon('oui-icon oui-icon-bandwidth_concept'),
minSearchItems: 0,
routeMatcher: new RegExp('^(/ip(/|$)|/vrack|/cloud-connect|(/network)?/iplb)'),
routeMatcher: new RegExp('^(/ip(/|$)|/network-security|/vrack|/cloud-connect|(/network)?/iplb)'),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just an improvement as it the same regex we can share it in another file
But not mandatory for this PR

@JacquesLarique JacquesLarique merged commit fa2ae98 into master Feb 29, 2024
@JacquesLarique JacquesLarique deleted the UXCT-527-network-security-dashboard-menu-bolder branch February 29, 2024 15:53
@JacquesLarique JacquesLarique mentioned this pull request Feb 29, 2024
14 tasks
JacquesLarique added a commit that referenced this pull request Feb 29, 2024
@JacquesLarique JacquesLarique restored the UXCT-527-network-security-dashboard-menu-bolder branch February 29, 2024 16:35
@JacquesLarique JacquesLarique deleted the UXCT-527-network-security-dashboard-menu-bolder branch February 29, 2024 16:59
JacquesLarique added a commit that referenced this pull request Feb 29, 2024
kqesar pushed a commit that referenced this pull request Mar 6, 2024
ref: UXCT-527

Signed-off-by: Stephanie Moallic <stephanie.moallic@corp.ovh.com>
kqesar pushed a commit that referenced this pull request Mar 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants