Skip to content

Conversation

@jeremystretch
Copy link
Member

Fixes: #19464

Remove the device initial data after instantiating the form to avoid flagging the actual device field as having changed data. (Ultimately we need to figure out a better way to pass contextual data under BulkEditView, but this suffices for now.)

@jeremystretch jeremystretch requested review from a team and arthanson and removed request for a team May 13, 2025 12:55
Copy link
Collaborator

@arthanson arthanson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is the device field needed here, on all the other components the device filed does not appear - we don't allow that to be changed on any other component bulk edit that I see?

@jeremystretch
Copy link
Member Author

@arthanson The device field was added to permit moving items from one device to another.

@arthanson arthanson self-requested a review May 13, 2025 17:22
@arthanson arthanson merged commit 3d1334a into main May 13, 2025
10 checks passed
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 13, 2025
@jnovinger jnovinger deleted the 19464-inventory-item-bulk-edit branch August 20, 2025 15:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bulk editing inventory items fails because no device is selected by default

3 participants