Skip to content
This repository has been archived by the owner on Aug 1, 2023. It is now read-only.

Icon updated as previous icon was misleading #504

Closed
wants to merge 2 commits into from
Closed

Icon updated as previous icon was misleading #504

wants to merge 2 commits into from

Conversation

Swap76
Copy link
Contributor

@Swap76 Swap76 commented Nov 7, 2019

This closes #18

Changes in the PR

  • The icon in public folder changed
  • The link for the icon chnaged

@CLAassistant
Copy link

CLAassistant commented Nov 7, 2019 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

moxiegirl pushed a commit to moxiegirl/animal-kingdom that referenced this pull request Nov 11, 2019
Merge stacks-archive#504
a540973 
Signed-off-by: Mary Anthony <[email protected]>
@moxiegirl moxiegirl mentioned this pull request Nov 11, 2019
Merged
@moxiegirl
Copy link
Contributor

@Swap76 You were trying to change the icon in the tab correct? Do this, you needed to create a favicon.ico a reference to a github image is not the best way to handle that. So, I carried and close your PR with #515

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Icon misleading
3 participants
@Swap76 @CLAassistant @moxiegirl