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

Ensure localized slug creates index on expected locale field #243

Closed
wants to merge 2 commits into from

Conversation

onomated
Copy link

@onomated onomated commented Jul 6, 2017

Index for slug is created on default locale slug sub-field when "localize: true" is specified. Otherwise, clients can specify a list of locales to create indices on.

… when "localize: true" is specified. Client can also specify a list of locales to create indices on.
Copy link
Collaborator

@dblock dblock left a comment

Choose a reason for hiding this comment

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

Code looks great to me. Cosmetic changelog/readme changes please, see Danger and my comments. Maybe add more documentation in README about how localization works and what indexes are being created, etc.

CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@onomated
Copy link
Author

onomated commented Jul 6, 2017

Sounds good. I'll make the changes here shortly

@dblock
Copy link
Collaborator

dblock commented Jul 6, 2017

Needs a rubocop -a ; rubocop --auto-gen-config, see CI.

@dblock
Copy link
Collaborator

dblock commented Apr 30, 2018

Bump @onomated, want to finish this?

@johnnyshields
Copy link
Member

I'll take this over.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants