-
Notifications
You must be signed in to change notification settings - Fork 3
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
Need the study ID in the study invitation #477
Comments
I don't really consider that a hack. If you look at the system decomposition, you see there is no dependency from the client to the studies subsystem, but only to the deployments subsystem. We'd need strong reasons to introduce tighter coupling here, even if just through an ID. What is the exact use case the client needs the study ID for? If this is an application-specific use case, it really doesn't belong in core. If the use case is somehow related to core, we may need some other feature or communication which is mediated by the deployments subsystem.
You can use a JSON object to store multiple fields in |
This is needed to get resources for the "study" (and not just the personal deployment).
That's because you're looking at the wrong dependency - the one in play here is the one linking to the "Reource" sub-system (which is implemented in CAWS). Here it says "study" in the description of the link.
The current use case is to get study-specific "resources" like the informed consent text, onboarding participant information, general info about the study, news articles about the study, etc.
Yes - we are also considering this if you don't want to fix this issue. |
That makes sense. That's indeed why a "resources" subsystem in core was expected, and as you know is currently still missing. :) I think the right solution here is to start working towards adding that subsystem to core.
The reason I am hesitant to add Similarly, the resources subsystem will likely benefit from being "studies" agnostic to keep things decoupled; at least, that was the original thinking behind the subsystem decomposition. If the client (through the deployments subsystem) receives resource IDs through the study protocol, there is no need to refer to Consider that when a client retrieves resources from a "resources" subsystem using Some of the cases where this is needed may be "application-specific", and such assumptions can be hardcoded for a specific platform (outside of core, but through extensibility of core), but others may be generic CARP core use cases.
It's not about wanting, it's about finding out whether it belongs in core. :) I.e., what is the benefit of adding it to core instead of |
Right now we are storing the study id in the
applicationData
of theStudyInvitation
. However, this is a hack and we would like to use application data for something else.Therefore,
studyId
needs to be a property of eitherStudyInvitation
orActiveParticipationInvitation
.The text was updated successfully, but these errors were encountered: