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

Fixed #6877 - Added notes to consumables, components on checkout #11650

Merged
merged 13 commits into from
Aug 10, 2022

Conversation

snipe
Copy link
Owner

@snipe snipe commented Aug 10, 2022

This adds a notes column to the pivot tables for checkouts for consumables and components and (only slightly) cleans up the UI to make it a little more consistent.

Screen Shot 2022-08-10 at 3 18 25 PM

Fixes #6877.

Signed-off-by: snipe <snipe@snipe.net>
Signed-off-by: snipe <snipe@snipe.net>
Signed-off-by: snipe <snipe@snipe.net>
Signed-off-by: snipe <snipe@snipe.net>
Signed-off-by: snipe <snipe@snipe.net>
Signed-off-by: snipe <snipe@snipe.net>
Signed-off-by: snipe <snipe@snipe.net>
Signed-off-by: snipe <snipe@snipe.net>
Signed-off-by: snipe <snipe@snipe.net>
Signed-off-by: snipe <snipe@snipe.net>
Signed-off-by: snipe <snipe@snipe.net>
@snipe snipe requested a review from uberbrady August 10, 2022 22:19
@snipe snipe changed the title Fixed #6877 add notes to consumables, components on checkout Fixed #6877 - Added notes to consumables, components on checkout Aug 10, 2022
Signed-off-by: snipe <snipe@snipe.net>
Copy link
Collaborator

@uberbrady uberbrady left a comment

Choose a reason for hiding this comment

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

I think this looks like some really good functionality that makes this part of Snipe-IT much more useful. I get a little confused by the Consumables having a 'notes' field, but each actual pivot ("consumption?") having its own 'note' field, but I think if we maintain that naming convention we should be in good shape. Thanks for putting this together!

app/Http/Transformers/ComponentsTransformer.php Outdated Show resolved Hide resolved
resources/views/components/view.blade.php Show resolved Hide resolved
Signed-off-by: snipe <snipe@snipe.net>
@snipe snipe merged commit c77d763 into develop Aug 10, 2022
@snipe snipe deleted the fixes/6877_add_notes_to_consumables branch August 11, 2022 03:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants