Skip to content

Sds updates#433

Merged
zwatson2001 merged 6 commits intomainfrom
sds-updates
Apr 22, 2026
Merged

Sds updates#433
zwatson2001 merged 6 commits intomainfrom
sds-updates

Conversation

@zwatson2001
Copy link
Copy Markdown
Collaborator

This PR makes a few small updates to SDS, adding an instruction screen before the first 5-card matching trial and preventing the user from continuing beyond a practice trial until they get it right.

@zwatson2001 zwatson2001 requested a review from asengupta3 April 7, 2026 01:22
@CCuskley CCuskley linked an issue Apr 8, 2026 that may be closed by this pull request

let fiveBlockIntroTrial: StimulusType;
let fiveBlockIntro: any;
if (taskStore().taskVersion === 2) {
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Is there a place where taskVersion is set to 2. Is there a way we manage it automatically?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

It's a url/variant param, so it's either set in the variant doc or passed in through the url if it's being run with the demo link. The plan is to set it to 2 for only the language variants that have translations for the new strings so that we can deploy these changes one language at a time.

Copy link
Copy Markdown
Collaborator

@asengupta3 asengupta3 left a comment

Choose a reason for hiding this comment

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

LGTM

@zwatson2001 zwatson2001 merged commit 46c6483 into main Apr 22, 2026
3 checks passed
@zwatson2001 zwatson2001 deleted the sds-updates branch April 22, 2026 16:17
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.

Something's the Same Spark Updates

2 participants