We currently only include a few if they authority types in the initial mappings - we need to round these out
Created attachment 64440 [details] [review] Bug 18825 - Elasticsearch - Update heading main-heading and see fields
I think there may be a couple of subfield lists that need to be fixed: - 110 should include b - 111 should not include b - 130 should not include c - 150 should be 150abgvxyz - 151 should be 151agvxyz - 155 should be 155avxyz
Created attachment 72809 [details] [review] Bug 18825 - Elasticsearch - Update heading main-heading and see fields
Updated, I don't think this is necessarily complete but I think we should just incrementally improve these as we can. Let me know if you have more suggestions
Created attachment 73007 [details] [review] Bug 18825 - Elasticsearch - Update heading main-heading and see fields Signed-off-by: Ere Maijala <ere.maijala@helsinki.fi>
I agree that further refinement can be done as necessary.
I want to push it along with bug 17373, we need it for 18.05. So pushing it soon may bring feedbacks.
Pushed to master for 18.05, thanks to everybody involved!