-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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 Matamo logo to website #3749
Conversation
Seems like the tests have been past. |
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.
Thanks. This issue was to update the site, not the README. 😉
Instructions for how to build the site can be found here.
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.
(requesting changes)
@boneskull Ohh thanks for pointing this out! I was under impression that README.md was supposed to be modified... Sorry about that. I was also wondering if I can compile it all-together in |
Description of the Change
Alternate Designs
Was asked to attach the logo at the bottom,, in a separate list. The logo is centered, though it can be relocated.
Why should this be in core?
N/A
Benefits
Mocha mentions its sponsor.
Possible Drawbacks
NA
Applicable issues
Fixes #3705
semver-patch