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

v8 update #144

Open
wants to merge 34 commits into
base: main
Choose a base branch
from
Open

v8 update #144

wants to merge 34 commits into from

Conversation

jadeddelta
Copy link
Collaborator

@jadeddelta jadeddelta commented Oct 17, 2024

this PR closes #138, focusing broadly on updating most (self paced reading is being completely reworked) plugins to jspsych v8, which necessitates breaking changes for all plugins as certain features handled by the core package like clearing timers/displays are left to the core package itself, along with certain plugins such as the audio plugins and plugins with button_html-esque fields being updated.

in order to improve developer experience, some plugins have added documentation where it was missing beforehand.

while all plugins function in v8, to move this PR off of draft and into merge consideration, there are several final passes and one blocking issue to content with:

  • final pass on documentation, making sure all parameters and data are documented
  • clarification on ParameterType.COMPLEX data types, specifically listing and documenting nested variables
  • changelog everything
  • this pr for letting json files be imported as modules, so that we can...
  • link package.json versioning to plugin info versioning

Copy link

changeset-bot bot commented Oct 17, 2024

🦋 Changeset detected

Latest commit: f02d18e

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

This PR includes changesets to release 25 packages
Name Type
@jspsych-contrib/plugin-video-several-keyboard-responses Major
@jspsych-contrib/plugin-image-array-keyboard-response Major
@jspsych-contrib/plugin-html-keyboard-response-raf Major
@jspsych-contrib/plugin-libet-intentional-binding Major
@jspsych-contrib/plugin-vsl-animate-occlusion Major
@jspsych-contrib/plugin-audio-multi-response Major
@jspsych-contrib/plugin-audio-swipe-response Major
@jspsych-contrib/plugin-html-keyboard-slider Major
@jspsych-contrib/plugin-image-multi-response Major
@jspsych-contrib/plugin-image-swipe-response Major
@jspsych-contrib/plugin-html-multi-response Major
@jspsych-contrib/plugin-html-swipe-response Major
@jspsych-contrib/plugin-nextcloud-filedrop Major
@jspsych-contrib/plugin-html-vas-response Major
@jspsych-contrib/plugin-vsl-grid-scene Major
@jspsych-contrib/plugin-survey-number Major
@jspsych-contrib/plugin-survey-slider Major
@jspsych-contrib/plugin-copying-task Major
@jspsych-contrib/plugin-corsi-blocks Major
@jspsych-contrib/plugin-html-choice Major
@jspsych-contrib/plugin-gamepad Major
@jspsych-contrib/plugin-ios Major
@jspsych-contrib/plugin-rdk Major
@jspsych-contrib/plugin-rok Major
@jspsych-contrib/plugin-pipe Minor

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
Member

@jodeleeuw jodeleeuw left a comment

Choose a reason for hiding this comment

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

Only reviewed a couple files as a demonstration of code review

@jadeddelta jadeddelta marked this pull request as ready for review November 3, 2024 17:50
Copy link
Member

@jodeleeuw jodeleeuw left a comment

Choose a reason for hiding this comment

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

I've only gotten through 1/3rd of the files so far. Here's my review in progress so far!

packages/plugin-copying-task/examples/example.html Outdated Show resolved Hide resolved
@@ -1,9 +1,9 @@
<!DOCTYPE html>
<html>
<head>
<script src="https://unpkg.com/jspsych@7.3.2"></script>
<script src="https://unpkg.com/jspsych@8.0.2"></script>
Copy link
Member

Choose a reason for hiding this comment

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

Is this the desired version? 8.0.2?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

it was the most recent version at the time of writing; though we could get around this by simply doing <script src="https://unpkg.com/jspsych@8"></script> to always query the latest version?

<script src="../dist/index.browser.js"></script>
<link rel="stylesheet" href="https://unpkg.com/jspsych@7.3.2/css/jspsych.css" />
<link rel="stylesheet" href="https://unpkg.com/jspsych@8.0.2/css/jspsych.css" />
Copy link
Member

Choose a reason for hiding this comment

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

Check version here too

packages/plugin-html-choice/src/index.ts Outdated Show resolved Hide resolved
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.

Update contrib plugins to version 8.0
2 participants