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.

Sign up for GitHub

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

Jump to bottom

fixes issue #5411 #5424

Closed
wants to merge 1 commit into from
Closed

fixes issue #5411 #5424

wants to merge 1 commit into from

Conversation

innith
Copy link
Member

@innith innith commented Sep 3, 2023

Fixes #5411

What changes did you make?

  • removed mention of CfA from the Brigade Organizer's Playbook list of partners and description.
  • replaced the partner and description details with the given details in _projects/brigade-organizers-playbook.md

Why did you make the changes (we will use this info to test)?

  • To keep the Website up-to-date and accurate.

Screenshots of Proposed Changes Of The Website (if any, please do not screen shot code changes)

Visuals before changes are applied

image

Visuals after changes are applied

image

@github-actions
Copy link

github-actions bot commented Sep 3, 2023

Want to review this pull request? Take a look at this documentation for a step by step guide!

From your project repository, check out a new branch and test the changes.

git checkout -b innith-remove-mention-of-CfA-from-Brigade-Organizers-Playbook-project-info-5411 gh-pages
git pull https://github.com/innith/website.git remove-mention-of-CfA-from-Brigade-Organizers-Playbook-project-info-5411

@github-actions github-actions bot added role: front end Tasks for front end developers role: back end/devOps Tasks for back-end developers P-Feature: Project Info and Page A project's detail page (e.g. https://www.hackforla.org/projects/100-automations) Complexity: Small Take this type of issues after the successful merge of your second good first issue size: 0.5pt Can be done in 3 hours or less ready for product labels Sep 3, 2023
@roslynwythe
Copy link
Member

The linked issue was not approved or prioritized and so the PR cannot proceed.

@roslynwythe roslynwythe closed this Sep 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aadilahmed aadilahmed Awaiting requested review from aadilahmed

@A-BMT02 A-BMT02 Awaiting requested review from A-BMT02

@aadi-singh01 aadi-singh01 Awaiting requested review from aadi-singh01

@93Belen 93Belen Awaiting requested review from 93Belen

@025Parag 025Parag Awaiting requested review from 025Parag

@a-t-klein a-t-klein Awaiting requested review from a-t-klein

@7kram 7kram Awaiting requested review from 7kram

@5amlim 5amlim Awaiting requested review from 5amlim

@1x55 1x55 Awaiting requested review from 1x55

Assignees
No one assigned
Labels
Complexity: Small Take this type of issues after the successful merge of your second good first issue P-Feature: Project Info and Page A project's detail page (e.g. https://www.hackforla.org/projects/100-automations) ready for product role: back end/devOps Tasks for back-end developers role: front end Tasks for front end developers size: 0.5pt Can be done in 3 hours or less
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

remove mention of CfA from Brigade Organizer's Playbook project info
2 participants
@innith @roslynwythe