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

[Mainnet]🐞 [Bug]: Rentable filter in node finder shows all dedicated nodes #2619

Closed
1 task done
khaledyoussef24 opened this issue Apr 28, 2024 · 8 comments
Closed
1 task done
Assignees
Labels
type_bug Something isn't working
Milestone

Comments

@khaledyoussef24
Copy link
Contributor

Is there an existing issue for this?

  • I have searched the existing issues

which package/s did you face the problem with?

Dashboard

What happened?

#2143

should not the rentable filter shows only nodes which allows to be rented (up and stand by only)

Steps To Reproduce

No response

which network/s did you face the problem on?

QA

version

2.4.0

Twin ID/s

No response

Node ID/s

No response

Farm ID/s

No response

Contract ID/s

No response

Relevant screenshots/screen records

Screenshot 2024-04-28 at 5 26 23 PM Screenshot 2024-04-28 at 5 26 26 PM

Relevant log output

na
@khaledyoussef24 khaledyoussef24 added the type_bug Something isn't working label Apr 28, 2024
@khaledyoussef24 khaledyoussef24 changed the title 🐞 [Bug]: Rentable filter in node finder shows all nodes 🐞 [Bug]: Rentable filter in node finder shows all dedicated nodes Apr 28, 2024
@khaledyoussef24
Copy link
Contributor Author

The rentable filter shows all nodes that are rentable and does not exclude the nodes that are down

show rentable should only show

  • up or stand-by nodes.
  • ready for renting nodes (not rented by another user).

@ramezsaeed ramezsaeed added this to 3.14.x May 1, 2024
@ramezsaeed ramezsaeed added this to the 2.6.0 milestone May 1, 2024
@Mahmoud-Emad
Copy link
Contributor

The rentable filter shows all nodes that are rentable and does not exclude the nodes that are down

show rentable should only show

  • up or stand-by nodes.
  • ready for renting nodes (not rented by another user).

While the reserve button disappear when the node is down it's not a problem, the user will discover only the node details, also, as a user and i have some dedicated nodes, i want to see all of them at some point

@khaledyoussef24
Copy link
Contributor Author

in that case the dedicated filter is already there.

Screenshot 2024-05-01 at 10 41 22 PM

@A-Harby
Copy link
Contributor

A-Harby commented May 12, 2024

Testnet, 2.4.0.

I tried to search for dedicated nodes, and with the up status, I found a few nodes, all of which were rentable.
image
Then I switched to rentable and up status, but I only found 1 node despite having more, as you can see in the picture above.
image

And the same happened on other status, so is it a bug ?
And the difference between dedicated and rentable is very confusing.

@AhmedHanafy725 AhmedHanafy725 removed this from 3.14.x May 29, 2024
@AhmedHanafy725 AhmedHanafy725 moved this to Accepted in 3.15.x Jun 26, 2024
@0oM4R 0oM4R self-assigned this Jul 3, 2024
@0oM4R 0oM4R moved this from Accepted to In Progress in 3.15.x Jul 3, 2024
@0oM4R
Copy link
Contributor

0oM4R commented Jul 3, 2024

@A-Harby resolved on #2960

Testnet, 2.4.0.

I tried to search for dedicated nodes, and with the up status, I found a few nodes, all of which were rentable. image Then I switched to rentable and up status, but I only found 1 node despite having more, as you can see in the picture above. image

And the same happened on other status, so is it a bug ? And the difference between dedicated and rentable is very confusing.

@0oM4R
Copy link
Contributor

0oM4R commented Jul 3, 2024

Issue update

work completed :
introduce new flow for this toggle:

  • if the Status filter is not selected ->> the status filter will be updated to Up and Standby
  • if the Status filter has any value -> nothing new happen, will keep it as it is
Screencast.from.03.2024.EEST.07.13.56.webm

@ehab-hassan what do you think about this flow?

@ehab-hassan
Copy link
Contributor

ehab-hassan commented Jul 4, 2024

As we discussed with @AhmedHanafy725 and @0oM4R, 'Rentable' means that the user needs to reserve an 'Up' or 'Standby' node. Therefore, let's freeze the 'Select Node Status' field to 'Up' and 'Standby' when the 'Rentable' toggle is active.

@0oM4R 0oM4R moved this from In Progress to Pending Review in 3.15.x Jul 4, 2024
@0oM4R 0oM4R moved this from Pending Review to In Verification in 3.15.x Jul 7, 2024
@khaledyoussef24 khaledyoussef24 changed the title 🐞 [Bug]: Rentable filter in node finder shows all dedicated nodes [Mainnet]🐞 [Bug]: Rentable filter in node finder shows all dedicated nodes Jul 14, 2024
@khaledyoussef24
Copy link
Contributor Author

verified on mainnet
Version 2.6.0
Rentable now when toggled is automatically shows dedicated nodes with status up and standby
Image
Image
Image
testcase

@khaledyoussef24 khaledyoussef24 moved this from In Verification to Done in 3.15.x Nov 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type_bug Something isn't working
Projects
Status: Done
Development

No branches or pull requests

6 participants