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

updated twitter logo #1405

Merged
merged 2 commits into from
Sep 4, 2023
Merged

Conversation

Prakhar-Shankar
Copy link
Contributor

Closes #1404
The logo has beed updated

Previously -
image

After changes-
image

Signed-off-by: Prakhar-Shankar <prakharshankar247@gmail.com>
@AtmegaBuzz
Copy link
Collaborator

LGTM, Thanks for the PR @Prakhar-Shankar

@AtmegaBuzz AtmegaBuzz requested a review from DonnieBLT September 3, 2023 06:27
@justary27
Copy link
Collaborator

@Prakhar-Shankar can you update your branch?

@DonnieBLT
Copy link
Collaborator

Yeah it seems to thrown an error when merging

@Prakhar-Shankar
Copy link
Contributor Author

Prakhar-Shankar commented Sep 4, 2023

I have updated it rn.
Please tell me if it is still not getting merged I will close this PR and create a new one.

@DonnieBLT DonnieBLT enabled auto-merge September 4, 2023 14:33
@DonnieBLT DonnieBLT merged commit 591620e into OWASP-BLT:main Sep 4, 2023
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.

Bug: Updating twitter logo in footer
4 participants