-
Notifications
You must be signed in to change notification settings - Fork 59
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
Add import for index mappings #145
Add import for index mappings #145
Conversation
Signed-off-by: Rupa Lahiri <[email protected]>
Signed-off-by: Rupa Lahiri <[email protected]>
Hi @prudhvigodithi , please review this PR. |
Hey @rblcoder thanks for your contribution, one question I have is since this logic is added to the Thank you |
@prudhvigodithi The read function is now able to read the mappings attribute from the underlying infrastructure. |
Thanks @rblcoder, LGTM. |
* Add import for index mappings Signed-off-by: Rupa Lahiri <[email protected]> * Format json using terrafmt Signed-off-by: Rupa Lahiri <[email protected]> --------- Signed-off-by: Rupa Lahiri <[email protected]>
@rblcoder Thank you. @prudhvigodithi any chance of getting a new release created with this fix? |
Description
Implement import of index mappings
Issues Resolved
#135 #71
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.