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

Bug on "Select all" button #72

Open
juannuciforo opened this issue Mar 16, 2023 · 13 comments
Open

Bug on "Select all" button #72

juannuciforo opened this issue Mar 16, 2023 · 13 comments

Comments

@juannuciforo
Copy link

juannuciforo commented Mar 16, 2023

hello! how are you?

this is the first commit that I do for a code that is not of my authorship, so I hope it is well done and adequate, I accept corrections! haha

The bug I currently have is that the script gets to capture all the people I follow correctly but never gets to enable the "select all" button so I right clicked it, I inspected it and removed the "disabled" attribute and it worked correctly so I found the way to solve it and I wanted to leave it so that if someone else happens to someone else, I can solve it this way,

thank you very much, excellent code that you were doing, I congratulate you.

@tomleb3
Copy link
Collaborator

tomleb3 commented Mar 17, 2023

Hey @juannuciforo
The "select all" checkbox is disabled until the scanning process finishes.
I suspect this points to an issue with the percentage calculation.
We've had a similar problem recently so it makes sense.

During the scanning process, there is a blue progress bar at the top of the page that disappears when the process reaches 100%.
Please let me know if it disappears in your case, or does it stay in place.

@juannuciforo
Copy link
Author

Hey @tomleb3

yes, I was looking at that bar, but it remains the same without completing even having finished the count of "followed" and the program stops analyzing so the only solution I found was to edit the code from the developers tool

@tomleb3
Copy link
Collaborator

tomleb3 commented Mar 17, 2023

I see.
So it is an issue with the percentage calculation, which is what needs to be fixed here.
I'll look into it.
Thanks!

@juannuciforo
Copy link
Author

Hey, it worked perfectly fine today,
idk why its working now but if you like I can close this issue

@tomleb3
Copy link
Collaborator

tomleb3 commented Mar 17, 2023

Let's leave it open for now.
If it didn't work at some point then there probably is a problem.
The percentage calculation likely needs a fixup.

@tomleb3
Copy link
Collaborator

tomleb3 commented Mar 20, 2023

Thanks for reporting. Issue is currently being looked into. @thefirst100yearsaredifficult

@m0rettini
Copy link

Same issue, tried on Opera Gx and Google chrome, i saw the percentage bar going but even after it calculated everyone it didnt reach 100%.

@thefirst100yearsaredifficult

Hello, issue seems to be fixed.

@AliRezaei-Code
Copy link

I have the same issue. Is it possible to remove the "disabled" attribute (make it selectable all the time) and instead display a message to the user to notify them of:

  1. The fact that they have to wait until the scan is complete
  2. A notification on the completion of the scan

Also the ability to be able to select all in the middle of the scan would be valuable.

@777paranoia
Copy link

Hello, issue seems to be fixed.

Here to report this issue is not fixed, still dealing with it

@bulutharbeli
Copy link

not fixed, i have the same issue :(

@RuanAragao
Copy link
Contributor

In my case at 98% the process stops, the following error is displayed in the console:

PtQ3PrOHBuX.js?_nc_x=Ij3Wp8lg5Kz:31 You are overriding current access token, that means some other app is expecting different access token and you will probably break things. Please consider passing access_token directly to API parameters

Do you also see the error?

@RuanAragao
Copy link
Contributor

I have the same issue. Is it possible to remove the "disabled" attribute (make it selectable all the time) and instead display a message to the user to notify them of:

  1. The fact that they have to wait until the scan is complete
  2. A notification on the completion of the scan

Also the ability to be able to select all in the middle of the scan would be valuable.

The problem is that because the items are sorted alphabetically, it ends up causing confusion when selecting during scanning.

I plan to implement controls in my testing, including stopping verification. Therefore, it is safer to stop scanning and select everyone.

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

No branches or pull requests

8 participants