Skip to content
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

[Documentation] Customization Guide: Models - fix #5457

Merged

Conversation

CoderMaggie
Copy link
Member

Q A
Doc fix? yes
New docs? no
BC breaks? no
Related tickets partially #5275
License MIT

@@ -86,6 +86,8 @@ The file should be placed in ``AppBundle/Resources/config/doctrine/Country.orm.y

3. Finally you'll need to override the model's class in the ``app/config/config.yml``.

Under the ``sylius_*`` where * is the name of the bundle of the model you are customizing, in our case it will be the ``AddressingBundle`` -> ``sylius_addressing``.
Copy link
Contributor

Choose a reason for hiding this comment

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

i think * should be in double back-ticks

Copy link
Contributor

Choose a reason for hiding this comment

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

In fact bundle's name is SyliusAddressingBundle, not AddressingBundle

@lchrusciel
Copy link
Member

It could be useful if would add a note with link to symfony2 documentation about doctrine and database migration. This section especially.

@CoderMaggie CoderMaggie force-pushed the docs/customization-guide-fix branch from 034ce62 to 7148d5b Compare July 7, 2016 09:14
@pjedrzejewski pjedrzejewski merged commit 3bc37f3 into Sylius:master Jul 7, 2016
@pjedrzejewski
Copy link
Member

Awesome, thanks Magda! 👍

@pjedrzejewski pjedrzejewski added the Documentation Documentation related issues and PRs - requests, fixes, proposals. label Jul 7, 2016
@Mr-Negative
Copy link

So, i'm stuck and so maybe it would be nice to add this to documentation. What about example of adding whole a entity relation? Adding variables is easy, but with relations between entities i'm missing probably one small step.

@pjedrzejewski
Copy link
Member

Definitely good idea! /cc @TheMadeleine We can add a relation to one of entities.

@CoderMaggie CoderMaggie deleted the docs/customization-guide-fix branch October 4, 2016 07:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation Documentation related issues and PRs - requests, fixes, proposals.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants