Skip to content
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

Child and past sessions data #18

Merged
merged 11 commits into from
Feb 21, 2024
Merged

Child and past sessions data #18

merged 11 commits into from
Feb 21, 2024

Conversation

okaycj
Copy link
Collaborator

@okaycj okaycj commented Feb 13, 2024

Summary

This PR adds two packages to our jspsych mono repo. Lookit-API will contain an implementation of Lookit's restful API and Lookit-helpers to provide functions to users to get access to child data and past session data for this child in this study.

Lookit-api

To help develop using the lookit restful API, I added a package to containerize the retrieval of data. This package should contain any implementation of a lookit endpoint. This package does not contain every available endpoint, just the ones needed for this project up to this point. At this time I have no intention of releasing this package to NPM.

The "initJSPysch" package needs to be updated using this package.

Lookit-helpers

Users are used to having access to child data and past sessions when running an experiment using EFP's generator. This package will give us functions to be used to expose these two data to a JSPsych experiment.

Copy link

changeset-bot bot commented Feb 13, 2024

🦋 Changeset detected

Latest commit: e8295e4

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@lookit/lookit-initjspsych Patch
@lookit/lookit-helpers Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link

@bleonar5 bleonar5 left a comment

Choose a reason for hiding this comment

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

looks great so far

@okaycj okaycj merged commit f528fb3 into main Feb 21, 2024
2 checks passed
@okaycj okaycj deleted the child-session-info branch February 21, 2024 15:30
readonly url?: string;
};

export interface Relationships {}
Copy link
Contributor

Choose a reason for hiding this comment

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

@okaycj could Relationships be something like this?

export interface Relationships = {
  links: Record<'related', string>
}

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Oh! I am not sure, but I am going to try this.

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.

3 participants