fix(dedicated.firewall): disable toggle button #10446
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Steffy29
requested review from
dectotam,
qpavy and
ghyenne
and removed request for
a team
Steffy29
force-pushed
the
UXCT-512-edge-network-firewall-update
branch
4 times, most recently
from
71072b2 to
540e997
Compare
...manager/apps/dedicated/client/app/ip/components/list/firewall/ip-ip-firewall-rule.service.js
Outdated
Show resolved
Hide resolved
dectotam
previously requested changes
Dec 5, 2023
...manager/apps/dedicated/client/app/ip/components/list/firewall/ip-ip-firewall-rule.service.js
Outdated
Show resolved
Hide resolved
Steffy29
force-pushed
the
UXCT-512-edge-network-firewall-update
branch
from
540e997 to
15ac05e
Compare
JacquesLarique
approved these changes
Dec 6, 2023
dectotam
previously requested changes
Dec 6, 2023
...manager/apps/dedicated/client/app/ip/components/list/firewall/ip-ip-firewall-rule.service.js
Outdated
Show resolved
Hide resolved
Steffy29
force-pushed
the
UXCT-512-edge-network-firewall-update
branch
from
15ac05e to
a24861f
Compare
Steffy29
force-pushed
the
feat/network-security-dashboard
branch
from
70dee84 to
9253589
Compare
Steffy29
requested review from
kqesar and
frenautvh
and removed request for
a team
github-actions
bot
added
the
has conflicts
Steffy29
force-pushed
the
feat/network-security-dashboard
branch
from
9253589 to
4f2fad5
Compare
github-actions
bot
added
translation required
and removed
has conflicts
Steffy29
force-pushed
the
feat/network-security-dashboard
branch
from
4f2fad5 to
c5e828e
Compare
Steffy29
force-pushed
the
feat/network-security-dashboard
branch
2 times, most recently
from
9a9d087 to
b396db4
Compare
github-actions
bot
added
has conflicts
Steffy29
force-pushed
the
UXCT-512-edge-network-firewall-update
branch
2 times, most recently
from
80d8ce1 to
344dc7b
Compare
ref: UXCT-512 disable toggle button if mitigation is on PERMANENT or FORCED status Signed-off-by: Stephanie Moallic <[email protected]>
Steffy29
force-pushed
the
UXCT-512-edge-network-firewall-update
branch
from
344dc7b to
227e1ab
Compare
|
Kudos, SonarCloud Quality Gate passed!
|
qpavy
approved these changes
Dec 12, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
developBreaking change is mentioned in relevant commitsDescription
disable toggle button if mitigation is on PERMANENT or FORCED status
Related