-
Notifications
You must be signed in to change notification settings - Fork 162
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 lastSearchInfo docs and unify interfaces with highlight directive- master #13286
Conversation
@onlyexeption @kdinev @mtsvyatkova @igdmdimitrov This caused breaking changes in a minor release btw. We used Here is the breaking change |
@PooSham I agree. @onlyexeption The breaking change was not checked in the checklist and I overlooked it during the review. Please add the property back in the |
@onlyexeption Actually, the PR should be reverted in |
@PooSham , 16.0.17 which was just released returns the matchInfoCache . We will probably revisit the API for 16.1 but it'll come with proper migrations. The change in a minor for 16.0 was not intended. |
Closes #13051
Additional information (check all that apply):
Checklist:
feature/README.MD
updates for the feature docsREADME.MD
CHANGELOG.MD
updates for newly added functionalityng update
migrations for the breaking changes (migrations guidelines)