Skip to content

Conversation

@narendravaghela
Copy link
Contributor

@narendravaghela narendravaghela commented Feb 28, 2017

Fixes #4687

@markstory markstory added this to the 3.x milestone Feb 28, 2017
@markstory markstory self-assigned this Feb 28, 2017
Installation using Oven
--------------------------------------------------

The another quick way to install CakePHP is `Oven <https://github.com/CakeDC/oven>`_.
Copy link
Member

Choose a reason for hiding this comment

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

"Another quick" or "The other quick".

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated!

--------------------------------------------------

The another quick way to install CakePHP is `Oven <https://github.com/CakeDC/oven>`_.
It is a simple PHP script which installs the CakePHP application skeleton, checks the
Copy link
Member

Choose a reason for hiding this comment

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

Aren't the system requirements checked first?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated the order.

* Upload the oven.php to your server directory.
* Execute the oven.php via your browser and click on the big oven icon.

Your CakePHP application is ready to go!
Copy link
Member

Choose a reason for hiding this comment

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

"After the installation completes, your ..."

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.


.. note::

IMPORTANT: This is not a deployment script, it's aimed to help developers installing
Copy link
Member

Choose a reason for hiding this comment

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

"script. it is aimed"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It was copied from https://github.com/cakedc/oven :)
Anyways, I have updated it.

@narendravaghela
Copy link
Contributor Author

@ravage84 Could you please review the updates?

Copy link
Member

@ravage84 ravage84 left a comment

Choose a reason for hiding this comment

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

Someone else shall merge. 👍 Thanks!

@ravage84 ravage84 changed the title Add reference to Oven - Fix #4687 Add reference to Oven Mar 1, 2017
@narendravaghela
Copy link
Contributor Author

Thanks 👍

installs the CakePHP application skeleton and sets the development environment up.

* Upload the oven.php to your server directory.
* Execute the oven.php via your browser and click on the big oven icon.
Copy link
Member

@dereuromark dereuromark Mar 1, 2017

Choose a reason for hiding this comment

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

Navigate to /path/to/oven.php via browser ...

maybe?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Member

Choose a reason for hiding this comment

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

Why duplicating if we can Link to it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

What about something like this -

  • Navigate to /path/to/oven.php via your browser e.g. http://localhost/cake/oven.php and click on the big oven icon.

or just

  • Navigate to /path/to/oven.php via your browser and click on the big oven icon.

Copy link
Member

Choose a reason for hiding this comment

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

I mean we already linked to it, I dont see the need to replicate the README instructions here.
We can omit this part, and start again with "After the installation completes".

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Agree 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants