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 New Icon: FilamentPHP #2120 #2121

Closed
wants to merge 4 commits into from

Conversation

AnshSinghSonkhia
Copy link

Added a New Icon for FilamentPHP

In response to the issue #2120

filament-plain-wordmark

About the FilamentPHP

  • Accelerated Laravel Development.
  • A collection of beautiful full-stack components. The perfect starting point for your next app.
  • Featured on GitHub Trending 99 times of all days.

Links and sources

website: https://filamentphp.com/

https://trendshift.io/?trending-language=php
https://trendshift.io/repositories/238

@canaleal
Copy link
Member

Hey @AnshSinghSonkhia, thank you for you interest in Devicon.

Unfortunately, the current pull request does not adhere to the required format. Please follow the steps outlined in the wiki on how to add a new Icon and create the PR.

Should you have any questions or need assistance, feel free to reach out. Thank you!

@AnshSinghSonkhia
Copy link
Author

Hey @canaleal,
I have created a new pull request at #2122 , based on the steps outlined in the wiki.

Kindly, help me merge my 1st contribution to our favourite Devicon!
Thank you!

@Snailedlt
Copy link
Collaborator

Hi @AnshSinghSonkhia, thank you for your contribution!
We are currently working on the release (#2105). Until the release is out we won't merge anything to development, and we will focus on getting the release out there :)

I also see that you've opened a new PR which superceeds this PR, so I closed this one. Comment if that was wrong, and I'll re-open it :)

@Snailedlt Snailedlt closed this Jan 30, 2024
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.

3 participants