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

[Feature Request]: Please add "Model No." field/column in "Assigned Assets" screen, Thank you! #15820

Closed
ma540x opened this issue Nov 14, 2024 · 4 comments

Comments

@ma540x
Copy link

ma540x commented Nov 14, 2024

Is your feature request related to a problem? Please describe.

Hello folks,
Feature request is not related to a problem, but it would absolutely be helpful if the screen where we can see assets assigned to us, would have that as an optional column to choose to display. We track devices and it would be awesome not just to see the Model (which does exist) but also the Model No field/column.

Describe the solution you'd like

If you could, please add "Model No." column as an available optional column option that users are able to choose on the "Assets assigned to [X]" page, URL --> http://something.com/account/view-assets

Pasting an image of what I would like to see as options in the Column drop down on that page, Thank you!

image

Describe alternatives you've considered

No response

Additional context

Thank you for all you do, Snipe it tool is awesome!

@snipe
Copy link
Owner

snipe commented Nov 14, 2024

Hi there - thanks for opening an issue :)

We track devices and it would be awesome not just to see the Model (which does exist) but also the Model No field/column.

That screen is only for items assigned to you yourself (or whoever the logged in user is). The model number fields is available throughout the rest of the admin screens. Is there possibly a use-case we're not considering here where you'd be tracking from the "view your own assets" page?

@ma540x
Copy link
Author

ma540x commented Nov 14, 2024 via email

@ma540x
Copy link
Author

ma540x commented Nov 14, 2024 via email

snipe added a commit that referenced this issue Nov 14, 2024
@snipe snipe closed this as completed in b4a2a4e Nov 14, 2024
@snipe
Copy link
Owner

snipe commented Nov 14, 2024

This is on master now

DarthKillian added a commit to DarthKillian/eti-assets that referenced this issue Dec 31, 2024
* Versioning stuff is hard - fixed v7.0.14 to v7.1.14

Signed-off-by: snipe <snipe@snipe.net>

* Versioning stuff is hard - fixed v7.0.14 to v7.1.14

Signed-off-by: snipe <snipe@snipe.net>

* this should work in theory - local is screwy though

* working except for null 🤔

* seems to work just fine now, needs translations

* note and rm dump and comment

* Specify table name in select

Signed-off-by: snipe <snipe@snipe.net>

* Add @maciej-poleszczyk as a contributor

* Fixed snipe#15812 - regression in action_type search

Signed-off-by: snipe <snipe@snipe.net>

* Removed Twitter links

Signed-off-by: snipe <snipe@snipe.net>

* Removed extra sort

Signed-off-by: snipe <snipe@snipe.net>

* Fixed anchor links

Signed-off-by: snipe <snipe@snipe.net>

* replace the via channel class

* removes tag if qr code is not present

* translations and attribute names

* fixed rest of dymos

* rm unnecessary imports

* Corrected route name for filestable on assets

Signed-off-by: snipe <snipe@snipe.net>

* Fixed delete model file route

Signed-off-by: snipe <snipe@snipe.net>

* oops, something went wrong translation

* Fixed snipe#15820 - added model number to view-assets

Signed-off-by: snipe <snipe@snipe.net>

* Additional fix for snipe#15817

Signed-off-by: snipe <snipe@snipe.net>

* Fixed snipe#15825

Signed-off-by: snipe <snipe@snipe.net>

* Fixed admin user not showing in download all activity report

Signed-off-by: snipe <snipe@snipe.net>

* this should be all it takes to fix this, i think

* replace env with config variable for from address

* Fix rollback for created_by

* Fix rollback for webhook_endpoint by using correct column type

* update default from address to alternate config variable

* Update README.md

typo mosrt to most under Join our Community links

* Feature: Added API endpoint for generating asset labels

* Feature: Added translations for label generation API endpoint

* Feature: Added API route for asset label generation

* Fixed variable name

Signed-off-by: snipe <snipe@snipe.net>

* Indicate requiredness on status select on bulk user checkin+delete

Signed-off-by: snipe <snipe@snipe.net>

* use existing validation strings

* add testing example var MAIL_FROM_ADDR

* fix mail.from.address variables in config, simplify envelop[ variable

* adding colon to password translation

* Add tests around component check ins via api

* Formatting

* Use === over ==

* Add tests around deleting component via ui

* Improve check for image existence

* Feature: Added category to asset checkout email

* Feature: Added category to asset checkin email

* Feature: Added category to asset acceptance/decline email

* Feature: Added category to asset requested email

* Fix: Removed setting validation as it is not strictly necessary

* no checked out to email doesnt trigger a 500

* Always run validation on checkin

* Use database file for sqlite in GitHub Action for tests

* remove the changes to mail.from.address default

* Add tests for component checkout api endpoint

* [Snyk] Upgrade jquery-ui from 1.14.0 to 1.14.1 snipe#15868

Signed-off-by: snipe <snipe@snipe.net>

* fixing colon placements

* Always call resizeCanvas

* WIP: move signature pad creation up and add debugging

* Remove debugging

* commit for testing

* orange bar for requirednessness

* Updated translations

Signed-off-by: snipe <snipe@snipe.net>

* Updated version

Signed-off-by: snipe <snipe@snipe.net>

* fix user api update from clearing location_id unnecessarily

* fixed mobile view of assets

* fix mobile view for asset and user

* Harden asset checkout validation by requiring ints

* requested changes from pr

* fixing line

* Dev assets

Signed-off-by: snipe <snipe@snipe.net>

* Use transformer on API update

Signed-off-by: snipe <snipe@snipe.net>

* Fixed order by notes for users

Signed-off-by: snipe <snipe@snipe.net>

* Fixed snipe#15901 - re-added required indicator on text and select custom fields

Signed-off-by: snipe <snipe@snipe.net>

* Fixed bulk user form in user detail view

Signed-off-by: snipe <snipe@snipe.net>

* Revert "Hardened asset checkout validation by requiring integer"

* Some CSS twiddling

Signed-off-by: snipe <snipe@snipe.net>

* Fixed funky layout in importer

Signed-off-by: snipe <snipe@snipe.net>

* fixes general webhook not firing

* Include MAIL_FROM_ADDR in phpunit configuration

* Fixed sr-only blank text

Signed-off-by: snipe <snipe@snipe.net>

* Harden asset checkout validation

* fix unaccepted assets report resend acceptance

* add property check

* adds mail from name to mail envelope

* fix typo

* layout / added category id

* fixed

* add mail class for unaccepted assets reminders

* fixed conditionals

* missed a spot

* fix duplicate emails being sent in unaccepted reminder command

* cleans up the Unaccepted Asset reminder variables

* wrap comment

* adds testing to unaccepted reminder command

* delete notification version of reminder

* Only update asset locations to assets checked out to users.

* Removed issue/PR templates

Signed-off-by: snipe <snipe@snipe.net>

* note, will come back to this once question is answered

* Show frontend "required" validation for model and status selects

* Updated supported versions

Signed-off-by: snipe <snipe@snipe.net>

* add null safe operator to acceptance reminder

* Fixes snipe#15952 - fixed typo in content-type

Signed-off-by: snipe <snipe@snipe.net>

* Add @sgross-emlix as a contributor

* Fixed snipe#15928 - updated method name to setCreatedBy from SetUserId

Signed-off-by: snipe <snipe@snipe.net>

* Fix league/commonmark's quadratic complexity bugs may lead to a denial of service

* Use appropriate category for licenses when seeding

* reversed order of the acceptance and note paramter

* fix other mailables

* Revert "Show frontend "required" validation for model and status selects"

This reverts commit 10a7ae8.

* fix other mailables

* Fix validation messages for select2 inputs

* leaving this here for notes etc

* Removed index and destroy settinga API endpoints

Signed-off-by: snipe <snipe@snipe.net>

* Fixes to 'clean' mode to better handle character sets and zero-values

* Check for assigned (not assigned_to) before trying to present() the name on bulk delete assets

Signed-off-by: snipe <snipe@snipe.net>

* Updated strings

Signed-off-by: snipe <snipe@snipe.net>

* Add accessory ui tests

* Add consumable ui tests

* Scaffold and implement some license checkin tests

* Bumped font-awesome to 6.7.0

Signed-off-by: snipe <snipe@snipe.net>

* Refactor of snipe#15235 - added accessory checkout to locations, assets

Signed-off-by: snipe <snipe@snipe.net>

* Use history icon

Signed-off-by: snipe <snipe@snipe.net>

* Added currency and history icons

Signed-off-by: snipe <snipe@snipe.net>

* Fixed variable in test

Signed-off-by: snipe <snipe@snipe.net>

* Updated user_id to created_by

Signed-off-by: snipe <snipe@snipe.net>

* Added new route

Signed-off-by: snipe <snipe@snipe.net>

* Updated route

Signed-off-by: snipe <snipe@snipe.net>

* Fixed user_id to created_by

Signed-off-by: snipe <snipe@snipe.net>

* No need to include assigned_to

Signed-off-by: snipe <snipe@snipe.net>

* Updated transformer

Signed-off-by: snipe <snipe@snipe.net>

* Removed create() method

Signed-off-by: snipe <snipe@snipe.net>

* Specify created_by in the API call as well

Signed-off-by: snipe <snipe@snipe.net>

* Added assignedAccessories method

Signed-off-by: snipe <snipe@snipe.net>

* Added assets endpoint

Signed-off-by: snipe <snipe@snipe.net>

* Added accessory transformer to assets transformer

Signed-off-by: snipe <snipe@snipe.net>

* Updated route in view

Signed-off-by: snipe <snipe@snipe.net>

* Updated routes with new endpoints

Signed-off-by: snipe <snipe@snipe.net>

* Updated comment

Signed-off-by: snipe <snipe@snipe.net>

* Added new test

Signed-off-by: snipe <snipe@snipe.net>

* Fixed admin -> adminuser property

Signed-off-by: snipe <snipe@snipe.net>

* Fixed property

Signed-off-by: snipe <snipe@snipe.net>

* Removed inadvertently added files.

* Add tests for license checkin

* Add permissions property to group factory

* Add simple tests to ensure views render

* Use correct id

* Remove leftover testing logic and fix checkout permissions

Removed console logs and temporary testing code from the bootstrap-table view. Updated the transformer to correctly check checkout permissions against the Asset class instead of PredefinedKit.

* rm note

* change barcode column names in seeder

* add note

* oops, rm dump

* Added a few order options on audit report

Signed-off-by: snipe <snipe@snipe.net>

* Update settings property keys

* Fix broken merge conflict files part 1

Signed-off-by: Darth Killian <rhuntington719@gmail.com>

* More Merge Conflict Bug Fixes

Signed-off-by: Darth Killian <rhuntington719@gmail.com>

* More bug fixes and add bulk checkout company & order number support back
in

Signed-off-by: Darth Killian <rhuntington719@gmail.com>

* Fix some additional lang stuff and disable user checkout on accessories

Signed-off-by: Darth Killian <rhuntington719@gmail.com>

* Fix some more missing language references

Signed-off-by: Darth Killian <rhuntington719@gmail.com>

* Fix broken language and testing files

Signed-off-by: Darth Killian <rhuntington719@gmail.com>

* Fix tests (again)

Signed-off-by: Darth Killian <rhuntington719@gmail.com>

---------

Signed-off-by: snipe <snipe@snipe.net>
Signed-off-by: Darth Killian <rhuntington719@gmail.com>
Co-authored-by: snipe <snipe@snipe.net>
Co-authored-by: spencerrlongg <spencer@spencerlong.com>
Co-authored-by: Godfrey M <godmartinz@gmail.com>
Co-authored-by: Marcus Moore <contact@marcusmoore.io>
Co-authored-by: victoria <vpak@grokability.com>
Co-authored-by: NebelKreis <leipold.johannes@web.de>
Co-authored-by: akemidx <kojotek.dx@gmail.com>
Co-authored-by: Brady Wetherington <bwetherington@grokability.com>
Co-authored-by: Joël Pittet <pittet@cs.ubc.ca>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants