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

Update MTL logo, social media #657

Merged
merged 5 commits into from
May 22, 2024
Merged

Conversation

oavramenko
Copy link
Contributor

@oavramenko oavramenko commented Apr 20, 2024

PyLadies Pull Request

Please document the following. We will aim to be as timely as possible in processing your pull request, however as this is a volunteer staffed project anticipate at least a week for pull requests to start to be reviewed.

Is this pull requesting adding or updating a PyLadies chapter website?

  • Yes
  • No

⚠️⚠️ If you marked yes, please close this pull request and head over to the PyLadies Chapter Websites repo to add your website. Additionally, you should remove your website code from this repository! ⚠️⚠️

Have you confirmed the website builds locally? See here.

  • Yes
  • No

Does your pull request pass the netlify/pyladies/deploy-preview check?

  • Yes
  • No

Pull Request Checklist

  • Fill out all sections above
  • Share pull request in PyLadies Slack channel #project-tech-infra

/cc @pyladies/tech-and-infra-admins

@oavramenko oavramenko requested a review from a team as a code owner April 20, 2024 15:05
@oavramenko oavramenko marked this pull request as draft April 20, 2024 15:20
Update linkedin class

Updated linkedin logo

Test linkedin logo
@oavramenko oavramenko marked this pull request as ready for review April 20, 2024 17:42
Copy link
Member

@BethanyG BethanyG left a comment

Choose a reason for hiding this comment

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

LGTM.

@BethanyG BethanyG requested a review from a team April 21, 2024 18:14
Copy link

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days

Copy link
Member

@willingc willingc left a comment

Choose a reason for hiding this comment

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

Thanks for the PR and your first contribution to the website. I like that you updated some of the URLs to use https. 💐

@willingc willingc merged commit 2100aa5 into pyladies:main May 22, 2024
7 checks passed
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.

None yet

3 participants