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

Do not use index prefix in settings files #15

Merged
merged 2 commits into from
Mar 21, 2018

Conversation

julienbourdeau
Copy link
Contributor

@julienbourdeau julienbourdeau commented Jan 19, 2018

Fix #14
Please comment on the issue.

@@ -86,8 +86,14 @@ protected function saveSettings($indexName)

protected function saveSynonyms($indexName)
{
<<<<<<< HEAD
Copy link

Choose a reason for hiding this comment

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

This looks like an accidental committed line?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oops ^^

@qrazi
Copy link

qrazi commented Feb 12, 2018

Any update on whether this PR will be merged?

@adiachenko
Copy link

Ping @julienbourdeau.

@qrazi
Copy link

qrazi commented Mar 5, 2018

@julienbourdeau Any change to merge this PR in? Would be very useful for current project for me already, separate from the proposed changes in #16 ... 😅

@julienbourdeau
Copy link
Contributor Author

The main reason is that because it's a BC break, I wanted to get more in the PR. But I think it has been long enough, I'll release this afternoon 🎉 👍

@qrazi
Copy link

qrazi commented Mar 20, 2018

@julienbourdeau Sorry to ping you again on this PR, but has something changed that prevents this PR from being merged? Anything I can help with?

@julienbourdeau
Copy link
Contributor Author

@qrazi Thanks for pinging me. I merge the other one and reworked this one to work with it. I'll run a few tests and tag the new version tomorrow if everything goes well.
Sorry again for the delay.

@julienbourdeau julienbourdeau merged commit 665a9a5 into master Mar 21, 2018
@julienbourdeau julienbourdeau deleted the feat/index-prefix-in-files branch March 21, 2018 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants