Skip to content
This repository has been archived by the owner on Jan 8, 2020. It is now read-only.

Updated the readme with the logo #7112

Merged
merged 3 commits into from
Jan 9, 2015
Merged

Updated the readme with the logo #7112

merged 3 commits into from
Jan 9, 2015

Conversation

macnibblet
Copy link
Contributor

Just noticed we don't have the zf logo in the zf2 repo readme

@@ -1,4 +1,5 @@
### Welcome to the *Zend Framework 2.3* Release!
![Logo](http://framework.zend.com/images/logos/ZendFramework-logo.png)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is the logo available in the repo?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Ocramius this is a good idea? In ZF1 is an external url (same url).. it isn't in the repo

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In ZF1 is an external url (same url).. it isn't in the repo

Also in zf-web and zf1-extras.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@froschdesign should be fixed imo, but isn't a big deal. I just want to do it right if we introduce it.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't add it in the repo

@Ocramius Ocramius self-assigned this Jan 9, 2015
@Ocramius Ocramius added this to the 2.3.4 milestone Jan 9, 2015
Ocramius added a commit that referenced this pull request Jan 9, 2015
@Ocramius Ocramius merged commit c88da3e into zendframework:master Jan 9, 2015
Ocramius added a commit that referenced this pull request Jan 9, 2015
@Ocramius
Copy link
Member

Ocramius commented Jan 9, 2015

Merged, thanks!

master: 67694e5
develop: ff0b5a4

Ocramius added a commit that referenced this pull request Jan 9, 2015
Ocramius added a commit that referenced this pull request Jan 9, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants