-
Notifications
You must be signed in to change notification settings - Fork 96
Typo on "cognitive walkthrough" card #27
Comments
Thanks @brittag! I think we can remove "a solution" but I'm not 100% sure and will defer to the team. :) I also notice that we may not need the comma in that sentence. |
Thanks for catching that @brittag. There are actually a couple areas on this method where I think we can clean the language up a bit. I'll file a pull request on it shortly. |
@jameshupp suggestions for a good way to make changes on the site & the printed cards, without losing track? Keeping the print cards up to date is definitely important, but it takes ~20 min each time we need to re-export and upload, so minimizing the number of times we do that would be 👍 |
@konklone does something similar for Pulse, he might have thoughts. |
Eagerly awaiting ^. In the meantime, @jenniferthibault I think we do a few things.
|
Oh! Question per ^: |
The how-to portion of this needed some edits to make it clearer. Most of it’s cosmetic, but #27 pointed out that the wording of step 4 was actually hard to understand, so I went ahead and made several edits while I was at it.
@wslack I do? :) Not sure what you mean. CCing @juliaelman. |
Best practices for managing a painful manual update process when data might be changing (which I think is - @konklone regularly updates it regularly each month) |
Oh, well, I run an update script that does all the work, but I do have to remember to trigger it every week, which I do with a calendar reminder. The updated results are versioned in GitHub week to week, and memorialized/managed with a pull request. It's not an awesome process, and something we hope to more fully automate later this year. |
With this commit both the digital copy and the card are on staging and ready to deploy. Moving discussion of how to handle the printed cards to #32. |
The "cognitive walkthrough" card says "Facilitate a walkthrough a solution where probing questions", which seems to be missing a colon (or another piece of punctuation or a word). Tiniest of issues!
The text was updated successfully, but these errors were encountered: