Skip to content
This repository was archived by the owner on Sep 16, 2021. It is now read-only.

first try to describe the alternate locales #691

Conversation

ElectricMaxxx
Copy link
Member

Q A
Doc fix? no
New docs? yes
Applies to SeoBundle
Fixed tickets #605

This will be my first try to describe the alternate locales. I know the english is bad, but it can be a base for the documentation i hope.

Tasks:

  • configuration page changes
  • grammar review
  • chapter in intro

=========================

The CMF provides a powerful way to persist document in different locales.
Each of those translated documents are Representations of another. In a
Copy link
Member

Choose a reason for hiding this comment

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

s/Representations/representations/

@dbu
Copy link
Member

dbu commented Jul 29, 2015

good start! i commented a bunch of details, but in general this is going in the right way.

i think we should be more clear that alternate locales is a concept, and the PHPCR-ODM provider just one possible implementation. (btw, with the RouteReferrersReadInterface, we could also build a provider based on that instead of the translations of the document in PHPCR-ODM. that would be a bit more generic - but not work for things like simple cms pages... out of scope for this PR anyways)

either i overlooked it or there is no explanation how the alternate locales finally end up in the html... i guess it happens with the twig function. but we should say this quickly and link to where we explain that twig extension.

@ElectricMaxxx ElectricMaxxx force-pushed the alternate_locale_handling branch from fba608a to 9111933 Compare July 31, 2015 14:32
@ElectricMaxxx ElectricMaxxx force-pushed the alternate_locale_handling branch from 9111933 to 904ca53 Compare August 9, 2015 05:09
'Acme\ApplicationBundle\AlternateLocaleProvider'
));

Now you have to configure ``CmfSeoBundle`` to use your custom alternate locale provider instead of the default one.
Copy link
Member

Choose a reason for hiding this comment

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

please break lines after 80 characters

@dbu
Copy link
Member

dbu commented Aug 22, 2015

i commented some things. having an example is ok, though i am not sure in this case how important it is. i think the important part is to explain how exactly things are meant to be used.

@ElectricMaxxx do you think you can update this again? otherwise i can have a go at this and try to formulate something.

@ElectricMaxxx
Copy link
Member Author

@dbu you know my english. Indeed it was my idea to just give the main input and hoped, that somebody can wrapp it up :-)

@dbu
Copy link
Member

dbu commented Aug 24, 2015

i will try to have a go at that tomorrow

@dbu dbu mentioned this pull request Apr 1, 2016
3 tasks
@dbu
Copy link
Member

dbu commented Apr 1, 2016

continued in #755

@dbu dbu closed this Apr 1, 2016
@lsmith77 lsmith77 removed the wip/poc label Apr 1, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants