Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Improve user query terms hits #1630

Merged
merged 1 commit into from
Jun 29, 2019
Merged

Improve user query terms hits #1630

merged 1 commit into from
Jun 29, 2019

Conversation

Martii
Copy link
Member

@Martii Martii commented Jun 29, 2019

  • Some new deps
  • This removes the HTML and MD from the about and stores in the limited _about.
  • Still currently testing 1KiB term max (combined)
  • Change default typing on _description to empty string.

NOTES:

  • This could also be achieved with marked and sanitize-html however giving this a whirl for source isolation.

Post #1629 and applies to #1548

@Martii
Improve user query terms hits
f1fa254 
* Some new deps
* This removes the HTML and MD from the `about` and stores in the limited `_about`.
* Still currently testing 1KiB term max *(combined)*
* Change default typing on `_description` to empty string.

NOTES:
* This could also be achieved with *marked* and *sanitize-html* however giving this a whirl for source isolation.

Post OpenUserJS#1629 and applies to OpenUserJS#1548
@Martii Martii added enhancement Something we do have implemented already but needs improvement upon to the best of knowledge. DB Pertains inclusively to the Database operations. migration Use this to indicate that it may apply to an existing or announced migration. DOC Pertains inclusively to the Documentation operations. CODE Some other Code related issue and it should clearly describe what it is affecting in a comment. labels Jun 29, 2019
@Martii Martii merged commit 122e186 into OpenUserJS:master Jun 29, 2019
@Martii Martii deleted the newDeps branch June 29, 2019 21:45
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 1, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
CODE Some other Code related issue and it should clearly describe what it is affecting in a comment. DB Pertains inclusively to the Database operations. DOC Pertains inclusively to the Documentation operations. enhancement Something we do have implemented already but needs improvement upon to the best of knowledge. migration Use this to indicate that it may apply to an existing or announced migration.
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@Martii