-
-
Notifications
You must be signed in to change notification settings - Fork 10
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
Garbage in tweet cause the sync to stop working #195
Comments
Have you found a way to fix this without all the content being reposted? |
I made a local patch but was not sure about the code so did not submit a PR. Maybe look at #201 |
No I tried to fix it but, well to cut a long story short it didnt work. Did you have a clever or quick way of even identifying the tweet that had the character? Just so that one can avoid using potentially problematic characters. |
Hello, I've been investigating this. I'm not able to actually reproduce it with the given tweet. Though, I've been able to identify potentially error prone case, so I made a fix for it. Let me know if the issue persist |
The error still seems to happen unfortunately. It is incredibly strange as I can not identify which posts are triggering this either. @jeffmaury how have you come along on this? 2024-10-09 19:27:21 TypeError: Cannot read properties of undefined (reading 'matchAll') |
A tweet with several links |
thanks, would make sense to reopen this potentially @louisgrasset , it has been quite erratic behavior. |
By the way, the potential fix is only on |
Describe the bug
A tweet causes the sync to stop working anymore
To Reproduce
The tweet causing the issue: https://x.com/jeffmaury/status/1829184894372884970
Expected behavior
An error message about parsing error and continue syncing
Screenshots
N/A
Additional supporting information
Windows11 Pro laptop
Using ce672e8
The text was updated successfully, but these errors were encountered: