repo sync #1142
Merged
repo sync #1142
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* new script to create webhook payload files for new version * version does not have to be a GHES version * remove newline
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
|
Thanks very much for contributing! Your pull request has been merged 🎉 You should see your changes appear on the site in approximately 24 hours. |
This was referenced Nov 9, 2020
This was referenced Nov 18, 2020
This was referenced Nov 28, 2020
This was referenced Dec 9, 2020
This was referenced Mar 31, 2021
Octomerger
pushed a commit
that referenced
this pull request
Nov 16, 2021
* search improvements * expose search result score on debug=1 qs (#22452) * Search index improvements (#22428) * concatenate headings for search ranking (#22445) * search index improvements * update index * add data-search property used to create search records * direct match title and normalize results to one per article * the search index is getting huge * add chinese lang, intro, and boost * create a article level record with no heading for each article * remove chinese language and exact match, boost breadcrumbs * break out debug mode into separate branch * break adding chinese out into another branch * one record per article remove limits * revert added spaces * revert adding property on article page * concatenate headings for search ranking * remove slug property, fix test * update record format in tests * revert adding this property back * scrape article-body not contents * adding heading property to tests * add headings to content property * Update script/search/parse-page-sections-into-records.js Co-authored-by: Peter Bengtsson <[email protected]> * Update script/search/parse-page-sections-into-records.js Co-authored-by: Peter Bengtsson <[email protected]> Co-authored-by: Peter Bengtsson <[email protected]> * search by popularity (#22496) * search by popularity * adjust factors based on length of query and normalize Lunr score * Apply suggestions from code review Co-authored-by: Rachael Sewell <[email protected]> Co-authored-by: Rachael Sewell <[email protected]> * turn html into text with appropriate spaces (#22528) * turn HTML into text with appropriate spaces Part of #1141 * fix unit test * improve tests * small refactor * slice the search results .content (#22514) * search by popularity * adjust factors based on length of query and normalize Lunr score * slice the search result .content Part of #1142 * Update lib/search/lunr-search.js Co-authored-by: Rachael Sewell <[email protected]> Co-authored-by: Rachael Sewell <[email protected]> * truncate translated search index records (#22475) * truncate translated search index records * page entire page object * revert search indexes * try reverting search indexes again... * out of sync with main in parent branch so reverting last commit again * rever deleted indexes * remove debug lines * correct package-lock.json * slice content string better (#22586) Part of #1149 * include home pages in the search (#22568) * Search remove hyphen as token (#22584) * slice better when not present in content (#22672) * fix missing search breadcrumbs (#22681) * fix missing search breadcrumbs * hopefully fix tests Co-authored-by: Peter Bengtsson <[email protected]> * Highlight based on combined matchData (#22692) * highlight based on combined matchData * refactoring * remove results when lang or version changes (#22697) * search with AND on title first and favor that more * commented out code * Update lib/search/lunr-search.js * only apply the wildcard to the title search if the last word is short Co-authored-by: Rachael Sewell <[email protected]>
jnidzwetzki
pushed a commit
to jnidzwetzki/docs
that referenced
this pull request
Oct 6, 2022
The usage of NSBP in a title breaks rendering
jnidzwetzki
pushed a commit
to jnidzwetzki/docs
that referenced
this pull request
Oct 6, 2022
The usage of NSBP in a title breaks rendering
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is an automated pull request to sync changes between the public and private repos.
🤖 This pull request should be merged (not squashed) to preserve continuity across repos, so please let a bot do the merging!