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

Removed twitter logo #366

Closed
wants to merge 4 commits into from
Closed

Removed twitter logo #366

wants to merge 4 commits into from

Conversation

Gowrish7
Copy link
Contributor

@Gowrish7 Gowrish7 commented Oct 4, 2023

Types of changes

fixes #365

  • Bug fix (non-breaking change which fixes an issue)
    Removed the twitter logo in the footer
    Thanks!
    image

Copy link
Member

@coliff coliff left a comment

Choose a reason for hiding this comment

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

  • changes need to be made in the src/ directory and make a build to update docs/index.html
  • Please remove twitter.svg image
  • Remove the twitter card meta tags (not just comment them out)

@coliff coliff marked this pull request as draft October 4, 2023 06:48
@Gowrish7
Copy link
Contributor Author

Gowrish7 commented Oct 4, 2023

  • changes need to be made in the src/ directory and make a build to update docs/index.html
  • Please remove twitter.svg image
  • Remove the twitter card meta tags (not just comment them out)

Can you help me with first comment i am new to open source and can you add hacktober label to if its possible now .

@Gowrish7 Gowrish7 marked this pull request as ready for review October 4, 2023 06:52
@coliff
Copy link
Member

coliff commented Oct 4, 2023

Hi. The details to make the build are in the README.

@Gowrish7
Copy link
Contributor Author

Gowrish7 commented Oct 4, 2023

Hi. The details to make the build are in the README.

Hey coliff i have made npm run serve and this is the screen shot
image

@Gowrish7
Copy link
Contributor Author

Gowrish7 commented Oct 4, 2023

@coliff please check whether the changes done are proper and please do specify if anything went wrong

@coliff
Copy link
Member

coliff commented Oct 4, 2023

Changes must be made in the src/ directory. Your PR doesn't include changes to the index file in the src directory. Please update and squash your commits.
P.S. sorry, we are not participating in Hacktoberfest.

@pragyamishra56
Copy link

Screenshot 2023-10-04 130749

Copy link
Contributor Author

@Gowrish7 Gowrish7 left a comment

Choose a reason for hiding this comment

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

i have done the changes in index.html in src directory can you check the latest commit

@pragyamishra56
Copy link

hey why have you done this I've been working on this issue

@Gowrish7
Copy link
Contributor Author

Gowrish7 commented Oct 4, 2023

hey why have you done this I've been working on this issue

Hey @pragyamishra56 its not yet merged you can still submit your PR . I am just learning how to contribute.
Thanks

@coliff
Copy link
Member

coliff commented Oct 4, 2023

I appreciate the enthusiasm for contributing from you both. :-)

@pragyamishra56
Copy link

Gowrish7

How is it possible you already created a PR
Screenshot 2023-10-04 132449

@pragyamishra56
Copy link

I appreciate the enthusiasm for contributing from you both. :-)

@coliff Could you kindly provide me guidance on How I request to pull the request so that I resolve it?

@Gowrish7 Gowrish7 closed this Oct 4, 2023
@Gowrish7 Gowrish7 reopened this Oct 4, 2023
@Gowrish7 Gowrish7 closed this Oct 4, 2023
@Gowrish7 Gowrish7 reopened this Oct 4, 2023
@Gowrish7
Copy link
Contributor Author

Gowrish7 commented Oct 4, 2023

I appreciate the enthusiasm for contributing from you both. :-)

@coliff Could you kindly provide me guidance on How I request to pull the request so that I resolve it?

@pragyamishra56 i guess you can make a seperate PR or should i close the PR ?

@pragyamishra56
Copy link

I appreciate the enthusiasm for contributing from you both. :-)

@coliff Could you kindly provide me guidance on How I request to pull the request so that I resolve it?

@pragyamishra56 i guess you can make a seperate PR or should i close the PR ?

No need to close the PR. Good luck with your next contribution. Maybe that's why, so keep growing and contributing to open source!

@coliff
Copy link
Member

coliff commented Oct 4, 2023

Please- let's just get this finished and merged. Please remove the twitter.svg and rebuild. Then squash the commits. thanks.

@pragyamishra56
Copy link

Please- let's just get this finished and merged. Please remove the twitter.svg and rebuild. Then squash the commits. thanks.

@coliff me or @Gowrish7

@coliff
Copy link
Member

coliff commented Oct 4, 2023

Please- let's just get this finished and merged. Please remove the twitter.svg and rebuild. Then squash the commits. thanks.

@coliff me or @Gowrish7

I really don't mind. Whoever can do it first.

@pragyamishra56
Copy link

pragyamishra56 commented Oct 4, 2023

can I do this? but how if he deletes his pr i can pull the request

@coliff
Copy link
Member

coliff commented Oct 4, 2023

can i do this ?

sure, if you want to make a new PR.

@roblarsen
Copy link
Member

Hi @pragyamishra56 use this as an opportunity to learn about how open source works. There are plenty

can I do this? but how if he deletes his pr i can pull the request

Just create a new PR. This is a GREAT way to learn about how GitHub and open source works. Consider that part homework to earn your Hacktoberfest t-shirt or whatever you're working towards.

@pragyamishra56
Copy link

pragyamishra56 commented Oct 4, 2023

I appreciate the enthusiasm for contributing from you both. :-)

@coliff Could you kindly provide me guidance on How I request to pull the request so that I resolve it?

@pragyamishra56 I guess you can make a separate PR or should I close the PR?

If you don't mind can you close your PR so that I can create pull request

@Gowrish7 Gowrish7 closed this by deleting the head repository Oct 4, 2023
@pragyamishra56 pragyamishra56 mentioned this pull request Oct 4, 2023
2 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.

remove Twitter Icon X
4 participants