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.

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

Feat/deidentification activ deactiv front refactoring #173

Merged

Conversation

jdcshug
Copy link
Collaborator

@jdcshug jdcshug commented Sep 8, 2021

  • refactoring deidentification
  • used setVisible instead of removing components
  • keep data modified when activate/deactivate deidentification
  • reset to empty values when saving with no deidentification

- if notification values are set to null or empty => set default values on activation notification or save destination
- remove clear values on enable/disable notification
- refactoring deidentification
- used setVisible instead of removing components
- keep data modified when activate/deactivate deidentification
- reset to empty values when saving with no deidentification
@sonarqubecloud
Copy link

sonarqubecloud bot commented Sep 8, 2021

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 8 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@nroduit nroduit merged commit 8e43612 into master Dec 1, 2021
nroduit pushed a commit that referenced this pull request Jan 22, 2025
… gather prepare and transfer of files

Merge in TELIMA/karnak from modify_forward_service to develop

* commit '11403c6b4b96091576dbad850d8c3fa8aa76b7ca':
  feat: modify ForwardService/multiple destinations: gather prepare and transfer of files
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DeIdentification front: refactoring + behaviour activate/deactivate deidentification
2 participants