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

[radar-appserver] Up appversion to 2.4.2 #240

Merged
merged 1 commit into from
Aug 2, 2024

Conversation

pvannierop
Copy link
Collaborator

This will add the following abilities to appserver:

  • File upload endpoint
  • Notifications vai email

Note

The respective values for helm were added in previous chart updates

@pvannierop pvannierop requested a review from keyvaann August 2, 2024 11:17
@pvannierop pvannierop self-assigned this Aug 2, 2024
@pvannierop pvannierop requested a review from Bdegraaf1234 August 2, 2024 11:18
@pvannierop pvannierop force-pushed the version/appserver-2.4.2 branch from fcc522c to 5690ae8 Compare August 2, 2024 11:24
Copy link

github-actions bot commented Aug 2, 2024

Great PR! Please pay attention to the following items before merging:

Files matching charts/*/values.yaml:

  • Is the PR adding a new container? Please reviewer, add it to the models (internal process)
  • Is the PR adding a new parameter? Please, ensure it’s documented in the README.md

This is an automatically generated QA checklist based on modified files.

@pvannierop pvannierop force-pushed the version/appserver-2.4.2 branch from 5690ae8 to a14d37a Compare August 2, 2024 11:26
@Bdegraaf1234 Bdegraaf1234 merged commit 8187b5c into main Aug 2, 2024
5 checks passed
@keyvaann keyvaann deleted the version/appserver-2.4.2 branch August 8, 2024 09:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants