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

Training Presentations and Other Supporting Materials #39

Merged
merged 4 commits into from
Oct 2, 2019

Conversation

bernars-usa
Copy link
Contributor

Addresses most of https://github.com/usds/OPM_pilot/issues/446 (all except RR training). Specifically, this adds the Day 1 and Day 2 job analysis workshop PPTs and the interview training PPT to the toolkits for those phases.

This also turns the Markdown versions of the sample competencies, proficiencies, and interview questions into a downloadable Word doc that can be printed and distributed at the job analysis workshop, as @stephaniefaith asked.

@bernars-usa
Copy link
Contributor Author

@humancompanion-usds - I can't see what's failing the build here, can you help?

@humancompanion-usds
Copy link
Contributor

Yeah, it's failing because of your change in pull request #38 where you added the email svg:

jekyll 3.8.5 | Error: Could not locate the included file 'svg/file-email.svg' in any of ["/work/site_repo/_includes", "/usr/local/rvm/gems/ruby-2.6.3/gems/jekyll-theme-primer-0.5.3/_includes"]. Ensure it exists in one of those directories and is not a symlink as those are not allowed in safe mode.

Let me pull down that branch and add the svg file. That should fix it.

@bernars-usa
Copy link
Contributor Author

it's failing because of your change in pull request #38

Ah, I didn't realize it would fail every PR until the one error was fixed in another branch. 😕

Copy link
Contributor

@humancompanion-usds humancompanion-usds left a comment

Choose a reason for hiding this comment

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

Materials, materials, materials. Looks good. Do we need to add these to the queue for OPM Policy? Or will they go in later? Just wondering if we need to keep a running list of diffs. If that is necessary, perhaps the change log would suffice? Perhaps make an update there next?

@bernars-usa
Copy link
Contributor Author

Do we need to add these to the queue for OPM Policy?

I don't believe so, as we're not seeking clearance for the website right now.

Just wondering if we need to keep a running list of diffs. If that is necessary, perhaps the change log would suffice? Perhaps make an update there next?

Yes, I'm going to make a single change log update for this PR and #38. 👍

bernars-usa added a commit that referenced this pull request Oct 2, 2019
@bernars-usa bernars-usa merged commit a0a8755 into master Oct 2, 2019
bernars-usa added a commit that referenced this pull request Oct 4, 2019
* Changelog updates for #38 and #39

* Today is the 3rd

* Add the small changes from #41
0kl-usds pushed a commit that referenced this pull request Oct 30, 2019
* Job Analysis Presentations

* Job Analysis Handout (Sample Comps and Interview Questions)

* Phone Interview Training

* Delete these, I don't know
0kl-usds pushed a commit that referenced this pull request Oct 30, 2019
* Changelog updates for #38 and #39

* Today is the 3rd

* Add the small changes from #41
@humancompanion-usds humancompanion-usds deleted the training-ppts branch March 3, 2020 18:11
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.

2 participants