-
Notifications
You must be signed in to change notification settings - Fork 25
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
Rename Fbits to Wake #16
Conversation
Hello! Our main header is "X-Powered-By": "Wake", that all of our sites got. So, even thought some old sites still got Fbits variable, they are already in the model header. In this case, we can delete Fbits json on the project, replacing it for Wake json.
|
Co-authored-by: Barry Pollard <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Our main header is "X-Powered-By": "Wake", that all of our sites got. So, even thought some old sites still got Fbits variable, they are already in the model header.
In this case, we can delete Fbits json on the project, replacing it for Wake json.
Co-authored-by: Rick Viscomi <[email protected]>
@Angeesantos can you also add the Wake logo to the icons directory? |
Hello Rick, Can you confirm if it's Wake logo or just an icon, because adding the logo it happens to get smaller and almost impossible to see it. So, if it can be Wake icon, like this file, I'll do it. |
Changed the icon name for wakecommerce.svg as the Wake icon it's already in the repository.
"cats": [ | ||
6 | ||
], | ||
"icon": "Fbits.png", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This file needed to be deleted.
I've opened a new PR for this at #19 since I'm not able to edit this branch.
Fixed in #19 |
Fixes #15 cc @Angeesantos