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

RSS svg icon #282

Merged
merged 2 commits into from
Jun 1, 2020
Merged

RSS svg icon #282

merged 2 commits into from
Jun 1, 2020

Conversation

mountainbug95
Copy link
Contributor

Fixes #278

@regisphilibert
Copy link
Member

@mountainbug95 this js file has been processed differently, any idea why?

@mountainbug95
Copy link
Contributor Author

@regisphilibert I'm not quite sure why it is different. I had to redeploy the assets to get my added class for the rss icon to work. I'm sure that's what caused it.

@regisphilibert
Copy link
Member

As the asset fingerpint is unchanged we don't need this file to be committed though. Any chance we can have it removed from the PR?

@mountainbug95
Copy link
Contributor Author

sure thing

@regisphilibert regisphilibert merged commit 624b79e into master Jun 1, 2020
Remicck pushed a commit to Remicck/gohugo-theme-ananke that referenced this pull request Mar 7, 2022
* Add rss icon to social icons

* Revert unnecessary asset deployment
@davidsneighbour davidsneighbour deleted the 278-rss-icon branch October 21, 2024 00:34
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.

RSS svg icon
2 participants