Visitar URL original
replace flexsearch-ts with orama and upgrade to next 14 by mrdrogdrog · Pull Request #5354 · hedgedoc/hedgedoc · GitHub
Skip to content

replace flexsearch-ts with orama and upgrade to next 14 - #5354

Merged
davidmehren merged 3 commits into
developfrom
orama
Jan 18, 2024
Merged

davidmehren merged 3 commits into
developfrom
orama

Conversation

@mrdrogdrog

Copy link
Copy Markdown
Member

Component/Part

Frontend cheatsheet

Description

This PR replaces flexsearch with orama

Steps

  • Added implementation
  • Added / updated tests
  • I read the contribution documentation and
    made sure that:
    • My commits are signed-off to accept the DCO.
    • This PR targets the correct branch: master for 1.x & docs, develop for 2.x

Related Issue(s)

Fix #5346

@mrdrogdrog mrdrogdrog self-assigned this Jan 17, 2024
@mrdrogdrog mrdrogdrog changed the title replace flexsearch-ts with orama replace flexsearch-ts with orama and upgrade to next 14 Jan 17, 2024

@ErikMichelson ErikMichelson left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

With compile error getting its own issue, this is fine

@mrdrogdrog

Copy link
Copy Markdown
Member Author

With compile error getting its own issue, this is fine

*Warning. If it would be an error it wouldn't compile successfully.

Signed-off-by: Tilman Vatteroth <git@tilmanvatteroth.de>
Signed-off-by: Tilman Vatteroth <git@tilmanvatteroth.de>
With node 18 the tests always crash.

Signed-off-by: Tilman Vatteroth <git@tilmanvatteroth.de>
@mrdrogdrog
mrdrogdrog enabled auto-merge (rebase) January 18, 2024 12:20
@mrdrogdrog
mrdrogdrog disabled auto-merge January 18, 2024 16:30
@mrdrogdrog
mrdrogdrog removed the request for review from DerMolly January 18, 2024 16:31
@davidmehren

Copy link
Copy Markdown
Member

For the record: The crash is nodejs/node#35889, fixed in Node 20 with nodejs/node#48510, backport to Node 18 is still open at nodejs/node#51004

@davidmehren
davidmehren merged commit 01d7eb9 into develop Jan 18, 2024
@davidmehren
davidmehren deleted the orama branch January 18, 2024 18:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Replace flexsearch with orama

3 participants