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

Mark all already loaded tweets as reported/bot #30

Open
eranshmil opened this issue Mar 11, 2019 · 0 comments
Open

Mark all already loaded tweets as reported/bot #30

eranshmil opened this issue Mar 11, 2019 · 0 comments
Labels
bug Something isn't working

Comments

@eranshmil
Copy link
Member

eranshmil commented Mar 11, 2019

Actual

  1. Go to some user's profile with at least two original tweets.
  2. Report on one of the tweets.
  3. You would be able to report again.
  • This is only a UX issue, the server ignores it.

Expected

The rest report buttons of his tweets should be disabled.

Just an idea:

Add an attribute to each checked user containing the userId, then just add the class to all the elements that have this attribute.
Should make the parsePosts and parseTweets simpler and we won't need to store the checked element inside the ObjectKeyMap variable.

@eranshmil eranshmil added the bug Something isn't working label Mar 11, 2019
@eranshmil eranshmil added ASAP! help wanted Extra attention is needed and removed help wanted Extra attention is needed labels Mar 11, 2019
@eranshmil eranshmil removed the ASAP! label Mar 13, 2019
@eranshmil eranshmil changed the title Report should mark all already loaded tweets Mark all already loaded tweets as reported/bot Mar 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant