Comments (9)
-
When asset is part of a kit and custody has been assigned individually (not via related Kit) I am unable to Release custody via Bulk Action. When I go to asset page I am able to Release custody. This should only not be possible when an asset is In custody because the entire kit was assigned custody if I am not mistaken
https://github.com/Shelf-nu/shelf.nu/assets/76112272/9fddef0e-33f2-44ef-bc55-5af1cd91ef49 -
I am unable to Assign custody to assets that are part of kit via Bulk select / actions.
https://github.com/Shelf-nu/shelf.nu/assets/76112272/15257dd7-9a4f-4c95-a47a-7cdd302ed3d7
Next one is kind of an edge case I think. I imported assets with kit and custodian resulting in 20 assets being part of a kit and having In custody status. However, all assets are In custody individually and not because the kit was assigned custody and assets inheriting the status. This can also happen without an import.
- When selecting the 20 assets that are in a kit and that have In custody status, Release custody action is blocked. For some reason when I go to next page (pagination) the action is possible
https://github.com/Shelf-nu/shelf.nu/assets/76112272/0c60cbf6-fe46-43b2-b34f-690c04e2dd2c
from shelf.nu.
Next one is kind of an edge case I think. I imported assets with kit and custodian resulting in 20 assets being part of a kit and having In custody status. However, all assets are In custody individually and not because the kit was assigned custody and assets inheriting the status. This can also happen without an import.
Yeah i know of this but tbh quite complex to solve. We ignore it for now. Also this is more related to imports.
@rockingrohit9639 can you address the other points please?
from shelf.nu.
@DonKoko this was on app.shelf
Bulk selected 2 bookings with Rerserved status
Performed Cancel bulk action
Got error but received automated mail for both booking saying they were cancelled
Refreshed page
Checked out 1 of 2 bookings
Bulk selected both bookings
Performed cancel bulk actions
Both bookings got cancelled and received automated mails saying they were cancelled again
from shelf.nu.
@rockingrohit9639 I have implemented a solution for that last point: #1170
from shelf.nu.
Okey so everything is resolved and released except this:
When selecting the 20 assets that are in a kit and that have In custody status, Release custody action is blocked. For some reason when I go to next page (pagination) the action is possible
It is quite important we fix this. As far as I understand this happens because we only have the ids and when you are on page 2 we don't have the remaining data. What I would recommend is to update the data structure. We currently store an array of ids.
We can do it in 2 ways:
- Array of objects - just store all the objects. Arrays can get a bit slow when they get huge so i am a bit skeptical about this
- Object of objects - Have an object where inside are objects with the key being the
item.id
and the value being theitem
. This is a pattern I have seen some frameworks use because its way more performant to access the data via the key, than having to use.find
or.filter
inside the array. This is my preferred option on how to handle it. The type of the collection will be something like this:
type Item = Asset | Kit | TeamMember | ....
type ItemsCollection = Record<Item['id'], Item>
I am curious to hear your opinion. @rockingrohit9639
from shelf.nu.
@jurrejansen @carlosvirreira this is all implemented and release.
@jurrejansen can you please test the feedback points you gave and if there is any problem please re-open this issue.
from shelf.nu.
@DonKoko I was running through these to do quick tests. And I experienced an issue with bulk cancelling bookings. After loading for quite a while...
- I tried cancelling 2 Reserved bookings which resulted in an error. However I do receive automated emails saying the bookings were cancelled
After checking out 1 of the bookings and trying again to bulk cancel it worked
https://github.com/user-attachments/assets/8b7c7856-66f7-4592-b7d1-c389058ec404
from shelf.nu.
@jurrejansen was this in production or on staging? Do I understand correctly that you received the email for both bookings but only the status of 1 of them got changed?
from shelf.nu.
@rockingrohit9639 I also found an issue:
I have filtered by in-custody and I have 2 pages. When I click select all the button is disabled, but that is not correct because actually all assets are in custody
from shelf.nu.
Related Issues (20)
- [Feature request]: Who performed last scan HOT 7
- [Feature request]: Improve Activity (Changes) automatic logging HOT 1
- [Feature request]: See all team members or link to them from admin dashboard
- Allow super admin to import assets for any workspace
- [Bug]: Camera continues scanning during redirect is happening HOT 1
- fix: don't prompt user to choose plan, when registering via a team workspace invite
- feature: build version validation
- Destroy all sessions on pwd reset HOT 6
- improvement: add kit column to booking PDF export HOT 7
- feat: add isEnterprise flag to CustomTierLimit
- [Feature request]: Implement "Self Service Plus" User Type with Enhanced Booking Capabilities HOT 4
- [Feature request]: Streamline Process for Linking Existing Assets to New QR Tags HOT 5
- [Bug]: Reported null
- chore: refactor ControlledActionButton to works immilar as to the bulk actions buttons
- [Improvement request]: Improve front-end of Asset page HOT 5
- [Feature request]: Improve UX of empty fields on asset pages HOT 1
- fix: bulk actions should be completely disabled for self service users HOT 1
- [BUG]: issues rendering long workspaces name on live
- [Bug]: I've set up a custom field of type Date, and when viewing an asset with this field set, it always shows one day earlier than I entered.
- [Bug]: Old Build Detected (Windows installable app)
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from shelf.nu.