-
Notifications
You must be signed in to change notification settings - Fork 194
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
Feature/select all on manage assets #1489
Feature/select all on manage assets #1489
Conversation
…ct-all-on-manage-assets
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just found 2 small things. Can you check them and then I can deploy for Carlos to also test.
…ct-all-on-manage-assets
…ct-all-on-manage-assets
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Man, i am sorry i found another issue i didnt notice before.
So here is the deal:
- Open manage assets for a booking that already has assets inside it
- Click select all
- Click clear all
- See issue - the selected assets number stays the at the initial number and it should be 0
I have temporarily placed the Select All button at the bottom-left of the modal. Please let me know where should I place it.
Issue - #1450