Skip to content

#306: Repository Layer for the Goals extension - #45

Merged
westei merged 56 commits into
developfrom
umm/306_goal-repository-layer
May 13, 2026
Merged

#306: Repository Layer for the Goals extension#45
westei merged 56 commits into
developfrom
umm/306_goal-repository-layer

Conversation

@westei

@westei westei commented Apr 2, 2026

Copy link
Copy Markdown
Member

This includes:

  • database tables to store goals data
  • Domain model for goals
  • Spring repositories for GoalTemplate, Goal and GoalConfiguration
  • Component tests for all Repositories

Rupert Westenthaler and others added 29 commits March 10, 2026 07:22
Defines

* study goal configuration including schedule for notifications as well as topics for goals
* API for GoalTemplates

Excludes:

* API for Goals - as it is unclear if Participant Goals are relevant for the StudyManager Frontend
…e in the compile-test pipeline configuration as those are not needed in the target repository
…ribution-2026-02

code contribution 2026 02 from UMM
…but being extensible for other application types.
…ndling

# Conflicts:
#	studymanager-services/src/main/java/io/redlink/more/studymanager/model/Participant.java
#	studymanager-services/src/main/java/io/redlink/more/studymanager/service/ParticipantService.java
#	studymanager-services/src/test/java/io/redlink/more/studymanager/service/ParticipantServiceTest.java
…gin-code-generator

MORE-Platform#469: Implemented application login token generation for participants
…on-access-bug

634 application access bug
…on-access-bug

#634: Small default pp url change
* added FIXME notes for things to change in a follow-up issue
* added the participantId to a log statement

Co-authored-by: Rupert Westenthaler <rupert.westenthaler@gmail.com>
…y-improvements

371 limesurvey improvements
…a dependency to the `LoginTokenService`. The `ApplicationAccessService` already cares about the necessary cleanups
…388_reset_tokens_after_preview

Prehab2rehab/388 reset tokens after preview
* SQL extension
* Repositories
* Component tests for Repositories

In general designed along similar existing entities
@westei
westei requested a review from janoliver20 April 2, 2026 05:35
janoliver20 and others added 25 commits April 7, 2026 09:06
…urvey-submit-date-fix

MORE-Platform#440: Fixed the lime survey submit date if empty
…mesurvey-data-storage

MORE-Platform#481: Reverted the limesurvey answer storage to the short id form
…-tags

#15: Implemented tags into the pipeline to correctly tag new versions…
…ce of the data gateway. Added the functionality also to the OccurredObservation repo & service of the study manager backend so that it is possible to write component tests!
…280_occurred-observation-by-start-end

MORE-Platform#280: OccurredObservation by start/end time
… observation and removed the token from being stored in the elastic
* Checks that only a single Survey is present for the Observation
* Checks that the `id` and `seed` data are present for the Observation

NOTE: Checks for actual Survey Data are not implemented as those would require API calls to the Lime Server and this is not feasible for periodic data health check. In addition their is no guarantee that a LimeSurvey is still available and unmodified on the server.
…esurvey-upgrade-fix

#24: Fixed an issue that prevented to change the lime survey id of an…
…479_data-health-lime-observation

MORE-Platform#479: DataHealth for LimeObservation
…ultiple studies and observations. It encodes the study, observation and participant in the givenname and adds more as lastname. It keeps backward compatibility for old/current studies by accepting the old format where the participantId was added as given and last name. It adds a UnitTest for the activate Method that checks if it performs the expected calls to the LimeServer and the SDK
Co-authored-by: Jan Cortiel <37823749+janoliver20@users.noreply.github.com>
…491_lime-survey-mapping-study-observation-participant

MORE-Platform#491: Multi Sudy/Observation LimeSurvey support
@github-actions

github-actions Bot commented May 6, 2026

Copy link
Copy Markdown

Test Results

197 tests  +87   196 ✅ +87   1m 55s ⏱️ +33s
 57 suites +21     1 💤 + 1 
 57 files   +21     0 ❌  -  1 

Results for commit c033872. ± Comparison against base commit 94fb470.

This pull request removes 1 and adds 88 tests. Note that renamed tests count towards both.
io.redlink.more.studymanager.component.observation.lime.LimeSurveyRequestServiceTest ‑ responseTest
io.redlink.more.studymanager.ApplicationTest ‑ contextLoads
io.redlink.more.studymanager.component.observation.lime.LimeSurveyObservationTest ‑ testActivation
io.redlink.more.studymanager.component.observation.lime.LimeSurveyObservationTest ‑ testValidation
io.redlink.more.studymanager.component.observation.lime.LimeSurveyRequestServiceTest ‑ fixNullDateTest
io.redlink.more.studymanager.controller.studymanager.ApplicationsAPIV1ControllerTest ‑ testGetStudyApplications
io.redlink.more.studymanager.controller.studymanager.AuditLogControllerTest ‑ testExportAuditLogs
io.redlink.more.studymanager.controller.studymanager.AuditLogControllerTest ‑ testGetAuditlogMetadata
io.redlink.more.studymanager.controller.studymanager.CalendarControllerTest ‑ testGetStudyTimeline
io.redlink.more.studymanager.controller.studymanager.ComponentControllerTest ‑ testComponentSpecificEndpointDoesNotExist
io.redlink.more.studymanager.controller.studymanager.ComponentControllerTest ‑ testComponentSpecificEndpointExists
…

♻️ This comment has been updated with latest results.

@janoliver20 janoliver20 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pom anpassen dann LGTM

Comment thread pom.xml Outdated
<project.reporting.outputEncoding>UTF-8</project.reporting.outputEncoding>

<repository.owner>redlink-gmbh</repository.owner>
<repository.owner>MORE-Platform</repository.owner>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Bitte anpassen

@westei
westei merged commit a137587 into develop May 13, 2026
5 checks passed
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