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

xmldom is not active anymore. #20

Closed
natlibfi-jonollil opened this issue Nov 22, 2019 · 2 comments
Closed

xmldom is not active anymore. #20

natlibfi-jonollil opened this issue Nov 22, 2019 · 2 comments
Assignees
@natlibfi-jonollil
Labels
enhancement New feature or request

Comments

@natlibfi-jonollil
Copy link
Contributor

https://www.npmjs.com/package/xmldom no updates in 3 years.

Possible replacement https://www.npmjs.com/package/fast-xml-parser ?
@natlibfi-arlehiko
Copy link
Contributor

natlibfi-arlehiko commented Nov 22, 2019 •
edited
Loading

Good catch! I'd go with xml2js since we're already using it elsewhere and it makes manipulating XML-like structures more enjoyable.

However, if fast-xml-parser is a drop-in replacement for xmldom we could go with that too.

So:

  • Prefer xml2js to fast-xml-parser if the code needs to be rewritten anyway
  • Either way, the new module must support both Node.js and browsers.

@natlibfi-arlehiko natlibfi-arlehiko added the enhancement New feature or request label Nov 22, 2019
@natlibfi-arlehiko
Copy link
Contributor

xmldom seems to be active again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@natlibfi-jonollil natlibfi-jonollil

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@natlibfi-arlehiko @natlibfi-jonollil