Skip to content

Conversation

nunomaduro
Copy link

@nunomaduro nunomaduro commented Oct 31, 2018

Q A
Bug fix? no
New feature? yes
BC breaks? no
Need Doc update no

Describe your change

This Pull Request adds the move method on the index. Usage:

index = client.init_index('name');
index.move('new_name')
index.get_settings() # Get the settings of the `new_name` index

@nunomaduro
Copy link
Author

Question for the team: Should I add the @deprecated tag into the client::move_index method?

@julienbourdeau
Copy link

Yes I think you can deprecate the move on client, I removed it from php v2 👌

@nunomaduro nunomaduro changed the title Adds move on the index Adds move_to on the index Nov 5, 2018
@nunomaduro
Copy link
Author

Closing this issue since it's not being implemented anymore.

@nunomaduro nunomaduro closed this Nov 8, 2018
@aseure aseure deleted the feat/add-move-on-index branch June 4, 2020 12:34
shortcuts added a commit that referenced this pull request Nov 24, 2023
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.

4 participants