Skip to content

brigade org playbook#1007

Merged
drubgrubby merged 2 commits intohackforla:gh-pagesfrom
matthewlee626:bop
Feb 14, 2021
Merged

brigade org playbook#1007
drubgrubby merged 2 commits intohackforla:gh-pagesfrom
matthewlee626:bop

Conversation

@matthewlee626
Copy link
Copy Markdown
Contributor

Fixes #965

@drubgrubby drubgrubby self-requested a review February 7, 2021 17:31
Copy link
Copy Markdown
Member

@drubgrubby drubgrubby left a comment

Choose a reason for hiding this comment

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

I haven't done a close inspection yet, but this looks good on first glance.

Once you fix the YAML exception you can push the changes and make a note here and in the issue that the changes have been fixed and we can check it and merge it.

In general, it's best not to create a pull request until you have completely solved the problem and you think the issue is ready to merge. Until then, you should just keep the comments and questions on the issue itself.

title: Brigade Organizers Playbook
description: Each of the 80+ Code for America Brigades is an experiment which generates valuable learnings and new effective processes and practices. However, Brigades and other civic tech volunteer organizations need not start from scratch. This project aims to improve existing structures and create new ones that make it easier to share replicable processes and practices so that organizers and members can iterate on each others work, improving outcomes for the whole network.
image: /assets/images/projects/brigade-organizers-playbook.png
alt: 'Brigade Organizer's Playbook Extension Project'
Copy link
Copy Markdown
Member

@drubgrubby drubgrubby Feb 7, 2021

Choose a reason for hiding this comment

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

(This has been edited from an earlier version. Bonnie suggested a diffefent fix.)

As you stated in the issue, this page is throwing a YAML exception.

The problem seems to be the apostrophe in line 6 - alt: 'Brigade Organizer's Playbook Extension Project'. It's trying to process the apostrophe within the string as part of the code and throwing a YAML Exception.

You can solve this by using double quotes outside the string:

The sentence would look like: alt: "Brigade Organizer's Playbook Extension Project"

@matthewlee626
Copy link
Copy Markdown
Contributor Author

Sorry about that! I'll keep that in mind for future issues.

@matthewlee626
Copy link
Copy Markdown
Contributor Author

Addressed the problem in the issue! This should be good to merge.

@drubgrubby drubgrubby merged commit 7572932 into hackforla:gh-pages Feb 14, 2021
@jbubar jbubar mentioned this pull request Feb 17, 2021
3 tasks
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.

Create project card: Brigade Organizers Playbook (BOP)

2 participants