Skip to content

Conversation

@gsaslis
Copy link
Contributor

@gsaslis gsaslis commented Jan 26, 2023

Note to reviewers:
the discord icon is not showing because it was only added to font-awesome @ 5.0.0

Upgrading font-awesome is something I don't feel comfortable doing, primarily because I don't have a fully working local env to test out the changes to the website. (https://github.com/softwarecrafters/website#-contributing-to-the-website is still "TBD")

the icon is not showing because it was only added to font-awesome @ 5.0.0

Upgrading font-awesome is something I don't feel comfortable doing, primarily because I don't have a fully working local env to test out the changes to the website.  (https://github.com/softwarecrafters/website#-contributing-to-the-website is still "TBD")
@gsaslis
Copy link
Contributor Author

gsaslis commented Jan 26, 2023

ah, cool, I just noticed there's a deploy preview here ! nicely done!!

@gsaslis
Copy link
Contributor Author

gsaslis commented Jan 26, 2023

I just noticed the discord invite is personalized ("yorgos invites you") and I was not aiming for that. I'll look into whether a vanilla one is possible.

@gsaslis
Copy link
Contributor Author

gsaslis commented Jan 26, 2023

I wasn't able to find an easy way of the personalized invite: https://support.discord.com/hc/en-us/articles/208866998-Invites-101

I don't mind if we keep that - I just don't like that it is so person-centric...

Thoughts / ideas are more than welcome!

@rradczewski
Copy link
Contributor

Regarding font awesome: please use an image for the logo, i won't have time soon to upgrade font awesome, i vaguely remember the license changing so it might not be trivial.

discord icon is only available on font-awesome 5.0.0+
@gsaslis
Copy link
Contributor Author

gsaslis commented Jan 31, 2023

@rradczewski should be all set now

this one will only grant temporary access. If folks don't accept the code of conduct (which happens through a discord role assignment, powered by carl-bot), they won't stay on the server.
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.

2 participants