Fix code typo and disable workflow on cron trigger in forks

[bskinn]

Deploy step only to occur on main branch, and on the openteamsinc repo. (NOTE: This has been changed, see below.)

Also update workflow to the current checkout action version.

Also fix a code typo that was breaking the workflow. (Unclear to me if execution reaching the code path where the typo was present means that something serious is going wrong with the queries.)

[bskinn]

Ok, wait -- is the live site getting deployed by the deploy step in this workflow?

Or by a separate github-pages deployment environment?

Or are both pieces important?

[bskinn]

<looks closer>

Looks like both pieces are important? The workflow step converts the built site into a Github Pages format in the gh-pages branch, and then Github Pages twigs off of that branch to actually publish to live?

[bskinn]

Oh, right -- cron trigger only runs on default branch, so that's redundant.

I still would like the filter to only run on the openteamsinc repo -- I don't need a regular build onto a gh-pages branch in my fork.

[bskinn]

Ok, changed my mind. We want to skip the entire workflow if the trigger is cronjob and the repo is anything other than openteamsinc/closember. Otherwise, we're making forks noisy, and wasting API calls.

[Carreau]

Thanks ! SorryI'm just back from CampQuansight.

[bskinn]

No worries, thanks! Not urgent.