Skip to content

removed breaked author comment name - #76

Open
CaiqueCoelho wants to merge 2 commits into
huaying:masterfrom
CaiqueCoelho:remove-author-comment
Open

removed breaked author comment name#76
CaiqueCoelho wants to merge 2 commits into
huaying:masterfrom
CaiqueCoelho:remove-author-comment

Conversation

@CaiqueCoelho

Copy link
Copy Markdown

Today this piece of code is broken:
author = browser.find_one(".FPmhX", els_comment).text
AttributeError: 'NoneType' object has no attribute 'text'
#75

Maybe the class with the author name from a comment has changed and any comment is geted once the code break before taking the comment and saving.

This solution is not the best, once I remove the author name, the code stop breaking and therefore the code keeps taking comments and saving. Perhaps this could be a solution at the moment and in the future we can get back to the author's name

@CaiqueCoelho

Copy link
Copy Markdown
Author

Hi @huaying can you check this PR please? 💃

@huaying

huaying commented Mar 14, 2020

Copy link
Copy Markdown
Owner

Sorry I am not maintaining this repo. If this is just a temporarily fix, please feel free to fork this repo and work on it, thanks.

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.

2 participants