Skip to content

feat(backend): support multiple organisms#567

Merged
fengelniederhammer merged 3 commits into
mainfrom
542-multi-pathogen-support-backend
Nov 22, 2023
Merged

feat(backend): support multiple organisms#567
fengelniederhammer merged 3 commits into
mainfrom
542-multi-pathogen-support-backend

Conversation

@fengelniederhammer

@fengelniederhammer fengelniederhammer commented Nov 21, 2023

Copy link
Copy Markdown
Contributor

resolves #542

@fengelniederhammer fengelniederhammer requested review from a user and JonasKellerer November 21, 2023 14:00
fengelniederhammer
fengelniederhammer commented Nov 21, 2023
View reviewed changes
Comment thread backend/src/main/kotlin/org/pathoplexus/backend/api/Organism.kt Outdated
@fengelniederhammer fengelniederhammer force-pushed the 542-multi-pathogen-support-backend branch from e2bd738 to e1ac667 Compare November 21, 2023 15:16
JonasKellerer
JonasKellerer suggested changes Nov 21, 2023
View reviewed changes

@JonasKellerer JonasKellerer left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great pr. Thanks. :) Mostly only naming issues.

Comment thread backend/src/test/resources/backend_config_single_segment.json Outdated
Comment thread backend/src/test/kotlin/org/pathoplexus/backend/controller/TestHelpers.kt Outdated
Comment thread ...end/src/test/kotlin/org/pathoplexus/backend/controller/SubmitReviewedSequenceEndpointTest.kt Outdated
Comment thread ...end/src/test/kotlin/org/pathoplexus/backend/controller/SubmitReviewedSequenceEndpointTest.kt Outdated
Comment thread backend/src/main/kotlin/org/pathoplexus/backend/controller/InfoController.kt Outdated
Comment thread backend/src/main/kotlin/org/pathoplexus/backend/service/DatabaseService.kt Outdated
Comment thread backend/src/test/kotlin/org/pathoplexus/backend/controller/ExceptionHandlerTest.kt Outdated
Comment thread backend/src/test/kotlin/org/pathoplexus/backend/controller/ExceptionHandlerTest.kt Outdated
Comment thread backend/src/test/resources/backend_config.json Outdated
@fengelniederhammer fengelniederhammer mentioned this pull request Nov 21, 2023
Closed
@fengelniederhammer fengelniederhammer force-pushed the 542-multi-pathogen-support-backend branch from 51fa990 to 5d614ad Compare November 22, 2023 09:51
@fengelniederhammer fengelniederhammer added the preview Triggers a deployment to argocd label Nov 22, 2023
@fengelniederhammer fengelniederhammer force-pushed the 542-multi-pathogen-support-backend branch from a1874f4 to e6eca15 Compare November 22, 2023 12:21
@fengelniederhammer fengelniederhammer merged commit 812ebf5 into main Nov 22, 2023
@fengelniederhammer fengelniederhammer deleted the 542-multi-pathogen-support-backend branch November 22, 2023 12:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@JonasKellerer JonasKellerer JonasKellerer approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

preview Triggers a deployment to argocd

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Multi-pathogen support: backend

2 participants

@fengelniederhammer @JonasKellerer