Skip to content

Biobank integration #23

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

Open
wants to merge 105 commits into
base: master
Choose a base branch
from
Open

Biobank integration #23

wants to merge 105 commits into from

Conversation

LBolzani
Copy link
Contributor

Good morning Peter, this branch include a feature to communicate with another service in order to make samples' request from Ada.

I remain at your disposal for any further information.

Luca

LBolzani added 23 commits June 21, 2021 16:57
# Conflicts:
#	server/build.sbt
# Conflicts:
#	web/app/org/ada/web/security/PacSecurityModule.scala
#	web/conf/core/auth.conf
# Conflicts:
#	web-ncer/app/controllers/orderrequest/BatchOrderRequestController.scala
#	web-ncer/app/controllers/orderrequest/BatchOrderRequestSettingController.scala
#	web-ncer/app/runnables/other/EmaRun.scala
@LBolzani LBolzani requested a review from peterbanda August 25, 2021 09:28
@peterbanda
Copy link
Member

Hey Luca,

It seems like none of the issues/comments from the original PR (#17) have been addressed. Pls. do so (for those still applicable) and then I will take a look.

Thanks,
:Peter

@LBolzani LBolzani closed this Aug 30, 2021
@LBolzani
Copy link
Contributor Author

Good morning Peter,

I'll try to update the code according to #17 .

I'll update you as soon as possible.

Luca

@LBolzani LBolzani reopened this Aug 30, 2021
@LBolzani
Copy link
Contributor Author

LBolzani commented Sep 2, 2021

Good morning Peter,

I've adapted the code according #17 pull request as much as possible.

Luca

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.

3 participants