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

Added Telegram Icon #677

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Added Telegram Icon #677

wants to merge 3 commits into from

Conversation

yeasin2002
Copy link

#658 requested Icon added

@SamCode011
Copy link

Interesting 📚

@yeasin2002
Copy link
Author

@SamCode011 Thanks, But unfortunately I think they are not going to accept this pull request.
From last two year they did not accepted any PR 😑
I did not notice that before.

@SamCode011
Copy link

No problem 😀👨🏻‍💻👍🏻

@oliver139
Copy link

@SamCode011 Thanks, But unfortunately I think they are not going to accept this pull request. From last two year they did not accepted any PR 😑 I did not notice that before.

But seems they recently reviewed the PRs and closed a batch of PRs.
However, they also left some of the PRs without merging, don't know why..

@SamCode011
Copy link

Thank uu @oliver139

@tandpfun
Copy link
Owner

tandpfun commented Jan 9, 2024

@oliver139 @SamCode011 as per the README, we don't accept pull requests on this repo for consistency purposes.

However, I sometimes go through and fix the issues and merge, but that doesn't happen often.

@yeasin2002
Copy link
Author

@tandpfun Would you merge this issue?

@nick747
Copy link

nick747 commented Feb 25, 2024

Thanks for adding it 🙌

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.

6 participants