Skip to content
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

Jump to bottom

IBX-4164: Fixed fill color for notifications icons #630

Merged
merged 1 commit into from
Nov 7, 2022
Merged

IBX-4164: Fixed fill color for notifications icons #630

merged 1 commit into from
Nov 7, 2022

Conversation

OstafinL
Copy link
Contributor

Question Answer
Tickets https://issues.ibexa.co/browse/IBX-4164
Bug fix? yes
New feature? no
BC breaks? no
Tests pass? yes
Doc needed? no
License GPL-2.0

Screenshot 2022-10-28 at 16 25 10

Checklist:

  • Coding standards ($ composer fix-cs)
  • Ready for Code Review

@OstafinL OstafinL added Bug Something isn't working Ready for review labels Oct 28, 2022
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@OstafinL OstafinL changed the base branch from main to 4.2 October 28, 2022 14:38
@dew326 dew326 merged commit 7388865 into 4.2 Nov 7, 2022
@dew326 dew326 deleted the IBX-4164-Notifications-wrong-color-of-x-icons branch November 7, 2022 10:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@GrabowskiM GrabowskiM GrabowskiM approved these changes

@dew326 dew326 dew326 approved these changes

@katarzynazawada katarzynazawada katarzynazawada approved these changes

@Gengar-i Gengar-i Gengar-i approved these changes

@tischsoic tischsoic Awaiting requested review from tischsoic

Assignees

@dew326 dew326

Labels
Bug Something isn't working QA approved
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@OstafinL @GrabowskiM @dew326 @katarzynazawada @Gengar-i