add: callFunction trial to call setUpPort after holdupMarkerTrial

[YUUU23]

• callFunction.js: JSPsych trial that calls function passed in
• startProcedure.js: calls buildCallFunctionTrial in callFunction.js passing in checkSerialPort() that access the electron process to call setUpPort() after holdUpMarkerTrial

[github-actions]

Visit the preview URL for this PR (updated for commit 29d7c01):

https://ccv-honeycomb--pr515-add-callfunction-35azeh52.web.app

_{(expires Wed, 07 Aug 2024 20:34:32 GMT)}

_{🔥 via Firebase Hosting GitHub Action 🌎}

_{Sign: 4ace1dcea913a952d2a1af84b94a4421bf36e610}

[RobertGemmaJr]

Looks fantastic! The structure of the code is excellent. Just a minor thing about how JS executes the callback functions and you'll be on your way!

[RobertGemmaJr]

Hey! Can you actually set this to be version ^1.1.0? jsPsych just released a new version and all of their plugins have been updated as well, we want the one that's major version #1

Suggested change

	"@jspsych/plugin-call-function": "^2.0.0",
	"@jspsych/plugin-call-function": "^2.0.0",

[RobertGemmaJr]

Awesome! I love the way you added it as a parameter

[YUUU23]

Thank you!!

[RobertGemmaJr]

Little oddness of JavaScript here! Because we're passing a callback function to the property func we actually need to supply a callback function, and not just the call to the function itself. There's a couple ways that we can do this:

Suggested change

	func: () => {
	func;
	},
	// 1) This way is best
	func: func,
	// 2) Or this would work too
	func: () => {
	func()
	},

Notice how with option 2 we cannot pass any parameters to the function - even if we define them in startProcedure.js! Because of this I would go with option 1, and then in startProcedure you pass a callback function to the code we actually want to execute:

procedure.push(buildCallFunctionTrial(() => { window.electronAPI.checkSerialPort() }));

[YUUU23]

Please see this PR #524 for the change of merging it into 3.4.2 instead!

[YUUU23]

Same code merged here #524