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.

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

Fix for complex html parsing #52

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

jywarren
Copy link
Member

@jywarren jywarren commented Mar 5, 2019

Fixes #51 (when complete!)

@jywarren
Copy link
Member Author

jywarren commented Mar 5, 2019

OK, working from this branch to try to do real HTML parsing: tautologistics/node-htmlparser#82

But getting: TypeError: undefined is not a constructor (evaluating 'new htmlparser.DefaultHandler') -- ok - tautologistics/node-htmlparser#57

@jywarren
Copy link
Member Author

jywarren commented Mar 5, 2019

OK fixed that but can't figure out how to recover the original HTML before it's parsed. May have to try another lib.

@jywarren
Copy link
Member Author

jywarren commented Mar 5, 2019

@jywarren
Copy link
Member Author

jywarren commented Mar 5, 2019

Somehow node-html-parser is breaking Jasmine, although it works normally in the browser. Left a note here asking for help! ashi009/node-fast-html-parser#4

@jywarren
Copy link
Member Author

Restarting the build to see if we get a different result... ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Error on some complex wiki pages
1 participant