Skip to content
This repository has been archived by the owner on Apr 21, 2021. It is now read-only.

Update the chapter extensions for Contao 4 #425

Open
wants to merge 8 commits into
base: 4.2
Choose a base branch
from
Open

Update the chapter extensions for Contao 4 #425

wants to merge 8 commits into from

Conversation

lionel-m
Copy link
Contributor

No description provided.

recommendations are available in the [Extension Repository][1] or in
[Packagist][2] (See "[Finding extensions or bundles][3]" chapter for more
detailed information). To create content, log into the back end and choose one
of the modules in the Content section of the navigation menu.
Copy link
Member

Choose a reason for hiding this comment

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

Lowercase C for content

`myExtensionName` with the name of your extension.

```php
new Contao\CoreBundle\HttpKernel\Bundle\ContaoModuleBundle('myExtensionName', $this->getRootDir()),
Copy link
Member

Choose a reason for hiding this comment

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

I think this line is obsolete if we have an example of AppKernel.php right below?


Run the command `php composer.phar require vendor/bundleName` in your
Copy link
Member

Choose a reason for hiding this comment

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

shouldn't that be composer require vendor/bundleName now?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Modified in 1c835f6


#### Manually

Find the extension you want to install in the [Extension Repository][1] and
Copy link
Member

@aschempp aschempp Oct 11, 2016

Choose a reason for hiding this comment

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

I would add an intro like this:

Almost all extensions for Contao 3 are also compatible with Contao 4. If a Contao 3 extension is not available on Composer, you can install it manually and try if it works with Contao 4. Be aware that you must also take care of installing all dependencies as listed in the Extension Repository.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added in 1c835f6

@aschempp
Copy link
Member

Looks RTM to me 👍
Want to add the french or a separate pull request?

@aschempp aschempp added this to the 4.2 milestone Oct 14, 2016
@lionel-m
Copy link
Contributor Author

I'll add the translation in this PR. 😃

@@ -132,5 +132,5 @@ release.
[3]: https://getcomposer.org/doc/00-intro.md#introduction
[4]: https://packagist.org/search/?q=&type=contao-bundle
[5]: https://packagist.org/search/?q=&type=contao-module
[6]: ../01-installation/installing-contao.md#symbolic-link
[6]: https://docs.contao.org/books/cookbook/en/folder-structure.html#symbolic-link
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@aschempp Initially, the link targeted the chapter "Installing Contao" but then we transferred this paragraph into the cookbook. Are you agree to add a cookbook chapter as a reference? Manual -> Cookbook

Copy link
Member

Choose a reason for hiding this comment

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

In general I'm fine with that, but in this case maybe we can just link to the Wikipedia article like the cookbook does? The sentence looks like an explanation what symbolic link is, and not about what Contao does with it (was described in the cookbook).

Copy link
Contributor Author

@lionel-m lionel-m Oct 18, 2016

Choose a reason for hiding this comment

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

I added the Wikipedia links 👍

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Modified in 06a85b6

@lionel-m
Copy link
Contributor Author

It's RTM for me too if you agree with contao/docs@7d55931#r83456222

@lionel-m
Copy link
Contributor Author

@aschempp: It is complete (RTM)

@aschempp
Copy link
Member

Would someone be available for the german translation? @fritzmg maybe? 😊

@fritzmg
Copy link
Contributor

fritzmg commented Nov 15, 2016

Sure, I'll have a look at it :)

content, log into the back end and choose one of the modules in the Content
section of the navigation menu.
recommendations are available in the [Extension Repository][1] or in
[Packagist][2] (See the chapter "[Finding extensions or bundles][3]" for more
Copy link
Contributor

Choose a reason for hiding this comment

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

Hm, this is only for Contao 4, right? Then the mention of the ER should be removed completely, I think?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We can write:

like articles, news, events, newsletters or forms. Further content types like banners, tickets, products or recommendations are available as bundles (See the chapter...


Contao 4 is built on top of the Symfony framework and takes advantage of its
functionalities but also of its terminology. In a Symfony project, an extension
is named a bundle.
Copy link
Contributor

Choose a reason for hiding this comment

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

I would rephrase this whole paragraph to:

Extensions are an essential part of Contao, because they allow you to add extra functionality. Contao 4 is based on the Symfony framework and takes advantage of its functionalities and its terminiology. In Contao 4 you can use classic extensions from previous Contao versions as well as so called Bundles.

@lionel-m
Copy link
Contributor Author

@aschempp, @fritzmg This PR is ready to be translated into German.

@aschempp
Copy link
Member

@fritzmg when could you take a look at this? 😊

@fritzmg
Copy link
Contributor

fritzmg commented Feb 27, 2017

Yes, sorry, I already started working on it but couldn't finish it yet. It's a little more effort than the other changes.

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.

3 participants