inm7/inm7-concepts
9
0
Fork 0
Code Issues 15 Pull requests 3 Projects Releases Packages Activity Actions

Adds parent_activity slot to StudyActivity #81

Closed
jsheunis wants to merge 1 commit from studyact into main
pull from: studyact
merge into: inm7:main
Conversation 2 Commits 1 Files changed 1 +13
jsheunis commented 2025-06-11 13:12:32 +00:00
Owner
Copy link

This is a simple change that would allow multiple activities to be grouped together as being part of a single StudyActivity. An example would be to group all DataItem-generating activities that happened during a single study "visit" together, with the single study visit being the parent activity.

This is a simple change that would allow multiple activities to be grouped together as being part of a single `StudyActivity`. An example would be to group all `DataItem`-generating activities that happened during a single study "visit" together, with the single study visit being the parent activity.
jsheunis force-pushed studyact from 4369c98a33
Some checks failed
Codespell / Check for spelling errors (pull_request) Successful in 22s
Details
Model checks / lint (pull_request) Failing after 1m26s
Details
Validate examples and verify unmodified conversion / lint (pull_request) Successful in 2m54s
Details
to b4fe03477c
All checks were successful
Codespell / Check for spelling errors (pull_request) Successful in 20s
Details
Model checks / lint (pull_request) Successful in 1m42s
Details
Validate examples and verify unmodified conversion / lint (pull_request) Successful in 2m49s
Details
2025-06-11 15:56:18 +00:00 Compare
mih commented 2025-06-12 06:55:48 +00:00
Owner
Copy link

What would speak against making is part_of? Now we have two properties with the semantics of part_of.

What would speak against making is `part_of`? Now we have two properties with the semantics of `part_of`.
jsheunis commented 2025-06-12 08:43:16 +00:00
Author
Owner
Copy link

Nothing that I can think of, happy to go with your option 👍

Nothing that I can think of, happy to go with your option 👍
mih closed this pull request 2025-06-12 09:03:38 +00:00
All checks were successful
Codespell / Check for spelling errors (pull_request) Successful in 20s
Details
Model checks / lint (pull_request) Successful in 1m42s
Details
Validate examples and verify unmodified conversion / lint (pull_request) Successful in 2m49s
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug

Something isn't working

  
documentation

Improvements or additions to documentation

  
duplicate

This issue or pull request already exists

  
enhancement

New feature or request

  
good first issue

Good for newcomers

  
help wanted

Extra attention is needed

  
invalid

This doesn't seem right

  
question

Further information is requested

  
wontfix

This will not be worked on

No labels
bug
documentation
duplicate
enhancement
good first issue
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
inm7/inm7-concepts!81
No description provided.