-
Notifications
You must be signed in to change notification settings - Fork 49
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
Unable to easily change the dashboard section order #1374
Comments
Hello, I’m interested in working on this issue. Could you confirm if it’s still available for someone to take? If you have any specific guidelines or suggestions for tackling this, please let me know. Thank you! |
@Zek1t0 no this is done. |
@adamsanadi6 include this commit in 10.2 branch as well since the branch has been cut |
Still we face the above mentioned issue QA Reference video : |
Reference Video : |
@dinesh2096 agreed during the first attempt also it should work as expected like happening during the second time. |
Remove unnecessary iteration of array and validate indexes are within range
Tests to be done:
|
lets write a unit test for this method. |
|
Issue:
Screen.Recording.2024-11-11.at.4.53.47.PM.mov
AC
The text was updated successfully, but these errors were encountered: