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

add ProFTPD icon #283

Merged
merged 3 commits into from Aug 27, 2021
Merged

add ProFTPD icon #283

merged 3 commits into from Aug 27, 2021

Conversation

SlrG
Copy link
Contributor

@SlrG SlrG commented Feb 20, 2021

resolves #282.

The permission from the original proftpd developer is linked in the icon request. Had to make it a little more bold than the example image in the request, to increase visibility.

fa_proftpd

@shinenelson shinenelson changed the title add proftpd icon, requested in #282 add ProFTPD icon Jul 10, 2021
Copy link
Member

@shinenelson shinenelson left a comment

Choose a reason for hiding this comment

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

Hey @SlrG, thank you for submission. I tried building the font and it looks fine.

The project is going though a refactor. There are some tasks that have been pending. Hence, it will be a while before we can merge and include this icon in Fork Awesome. But we hope it to be very soon 🚀

@SlrG
Copy link
Contributor Author

SlrG commented Jul 12, 2021

That's great news. Thank you very much for info. 👍

@shinenelson shinenelson merged commit 614dfab into ForkAwesome:master Aug 27, 2021
@shinenelson
Copy link
Member

@all-contributors please add @SlrG for design

@allcontributors
Copy link
Contributor

@shinenelson

I've put up a pull request to add @SlrG! 🎉

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.

Icon Request: fa-proftpd
2 participants