Skip to content

#497 limesurvey datahealth lastpage#498

Merged
westei merged 2 commits into
developfrom
prehab2rehab/497-limesurvey-datahealth-lastpage
May 7, 2026
Merged

#497 limesurvey datahealth lastpage#498
westei merged 2 commits into
developfrom
prehab2rehab/497-limesurvey-datahealth-lastpage

Conversation

@westei
Copy link
Copy Markdown

@westei westei commented May 7, 2026

This changes the data health validation for lime survey from seed to lastpage see #497 for more information why this change is necessary

Rupert Westenthaler added 2 commits May 7, 2026 15:03
…n validation for all data types expect OBJECT
…tpage` property instead of `seed`. Updated the implementation so that the validation of INTEGER type properties is now supported by the `QuestionObservationUtils`. Adapted the UnitTest to provide Mock data for `lastpage` instead of `seed`
@westei westei requested a review from janoliver20 May 7, 2026 13:06
@westei westei self-assigned this May 7, 2026
Copy link
Copy Markdown
Contributor

@janoliver20 janoliver20 left a comment

Choose a reason for hiding this comment

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

LGTM

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 7, 2026

Test Results

187 tests  ±0   186 ✅ ±0   1m 53s ⏱️ -4s
 54 suites ±0     1 💤 ±0 
 54 files   ±0     0 ❌ ±0 

Results for commit b1caa42. ± Comparison against base commit cbc97b1.

@westei westei merged commit e459cfe into develop May 7, 2026
5 checks passed
@westei westei deleted the prehab2rehab/497-limesurvey-datahealth-lastpage branch May 7, 2026 13:12
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