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

change location to status on assets table header #74

Closed
wants to merge 1 commit into from

Conversation

peelman
Copy link

@peelman peelman commented Nov 27, 2013

No description provided.

@snipe
Copy link
Owner

snipe commented Nov 27, 2013

When an asset is checked out, it by definition means it's assigned to a user, which is why it's tied to that user's location.

@peelman
Copy link
Author

peelman commented Nov 27, 2013

OK, but we have users who maintain two offices in separate physical locations. They have equipment assigned to them in both (IP phones for starters, but monitors, docking stations, etc would also fall into this).

There's also the question of servers; servers don't have an "owner", at least not in our org unit. Even if we stretched and assigned them one based on their use/purpose, we have servers at a Colo, a remote facility, etc., network gear too.

The location model (as it is currently) breaks down in all but the simplest cases. But that's more or less outside the scope of this pull request and probably not going to get solved by this :)

@peelman peelman closed this Nov 27, 2013
@ghost ghost mentioned this pull request Jun 9, 2016
2 tasks
@veen1981 veen1981 mentioned this pull request Jul 7, 2016
2 tasks
This was referenced Jul 19, 2016
@dooley1023 dooley1023 mentioned this pull request Feb 2, 2017
2 tasks
@katecruser katecruser mentioned this pull request Feb 8, 2017
2 tasks
@Mike1704 Mike1704 mentioned this pull request Mar 3, 2017
@ESWBitto ESWBitto mentioned this pull request Mar 16, 2017
2 tasks
@Mikey84 Mikey84 mentioned this pull request Oct 22, 2020
2 tasks
@LeeSharman LeeSharman mentioned this pull request Feb 26, 2021
@ecmcidev ecmcidev mentioned this pull request Apr 6, 2022
2 tasks
@cookierun22 cookierun22 mentioned this pull request Jan 28, 2023
2 tasks
@Bjufen Bjufen mentioned this pull request Feb 9, 2023
2 tasks
@jayavman jayavman mentioned this pull request Sep 2, 2023
2 tasks
@uglycrab uglycrab mentioned this pull request Mar 27, 2024
2 tasks
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

Successfully merging this pull request may close these issues.

2 participants