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

Add Localize Helpers #62

Merged
merged 2 commits into from
Dec 3, 2015
Merged

Add Localize Helpers #62

merged 2 commits into from
Dec 3, 2015

Conversation

dlitvakb
Copy link
Contributor

No description provided.

@dlitvakb dlitvakb force-pushed the dl/add-localize-helper branch 2 times, most recently from e487d15 to fdde1ba Compare November 23, 2015 20:19

If your fields are not localized, the value of the field will be returned.

In case of the field being localized but the locale not found, it will use
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 we should reformulate this to

In case of the field being localized, but no value being set for a given entry, it will use…

Otherwise people might be confused what "not found" means and why it would happen.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This has been changed

@dlitvakb
Copy link
Contributor Author

dlitvakb commented Dec 2, 2015

This PR should not yet be merged

contentful/contentful.rb#73 fix will break this, working on a solution

@dlitvakb
Copy link
Contributor Author

dlitvakb commented Dec 2, 2015

Specs will be green when contentful/contentful.rb#78 is merged and released

neonichu added a commit that referenced this pull request Dec 3, 2015
@neonichu neonichu merged commit b7e2dfb into master Dec 3, 2015
@neonichu neonichu deleted the dl/add-localize-helper branch December 3, 2015 15:15
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.

None yet

2 participants