Skip to content
This repository has been archived by the owner on May 16, 2023. It is now read-only.

Add new app config parameters #1749

Merged
merged 12 commits into from
Mar 9, 2022
Merged

Conversation

AliceBilc
Copy link
Contributor

@AliceBilc AliceBilc commented Feb 28, 2022

Checklist

  • Test your changes as thoroughly as possible before you commit them. Preferably, automate your test by unit/integration tests.
  • Make sure mvn -P integration-test clean verify runs for the whole project and, if you touched any code for a service in the services folder, ensure it can be run with spring-boot:run

Description

@hilmarf hilmarf added this to the 2.20 milestone Feb 28, 2022
@AliceBilc
Copy link
Contributor Author

@mlenkeit, we're already using this parameter in the filter method https://github.com/corona-warn-app/cwa-server/blob/main/services/submission/src/main/resources/application.yaml#L57
Do you think it would be a good ideea to change here the name and the value?

@AliceBilc AliceBilc marked this pull request as ready for review March 2, 2022 07:25
@AliceBilc AliceBilc requested review from a team, T-Bendrath and ioangut March 2, 2022 07:25
@hilmarf hilmarf requested a review from mlenkeit March 4, 2022 12:32
@ioangut ioangut enabled auto-merge (squash) March 9, 2022 08:12
@sonarcloud
Copy link

sonarcloud bot commented Mar 9, 2022

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

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@ioangut ioangut merged commit 6236ee7 into main Mar 9, 2022
@ioangut ioangut deleted the feature/restrict-exposure-history branch March 9, 2022 08:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants