jmcgover / unblock-all

A javascript script that unblocks all of a Twitter user's blocked Twitter accounts
MIT License
30 stars 20 forks source link

Small issue, plus comments #4

Open a-einstein opened 5 years ago

a-einstein commented 5 years ago

Thanks for sharing this, even after a couple of years this still helped me to clean up after reporting a lot of accounts.

Small issue: a missing ';' at the end of the line of 'var unblockButton'.

Update for UNBLOCKING, the line should now be: var unblockButtons = document.getElementsByClassName("unblock-text");

Addition for UNMUTING, change the line to: var unblockButtons = document.getElementsByClassName("unmute-user-item");

I refreshed the page a couple of times because of the great number of items, then re-pasted the script in , before calling main() again.

DocRedFox commented 5 years ago

I made the changes you suggested to the code but it appears that twitter limits the number of requests if your block list is long. I'm thinking the while loop at the end needs to be adjusted so the client isn't making too many requests? Any idea on how to modify that to get this to still work?

Sanjana-Rao commented 4 years ago

Thanks for sharing this, even after a couple of years this still helped me to clean up after reporting a lot of accounts.

Small issue: a missing ';' at the end of the line of 'var unblockButton'.

Update for UNBLOCKING, the line should now be: var unblockButtons = document.getElementsByClassName("unblock-text");

Addition for UNMUTING, change the line to: var unblockButtons = document.getElementsByClassName("unmute-user-item");

I refreshed the page a couple of times because of the great number of items, then re-pasted the script in , before calling main() again.

I made the changes but it only says "unblock 0 accounts" It is scrolling but I don't think it is recognising the block button. Please help?