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

Added: Location column in 'Assets Assigned' View. #15494

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

swift2512
Copy link

Description

Adds location information in 'Assets Assigned' View. This may be useful for the users, who have assets in different locations. (They work from home, for e.g.)

image

Copy link

what-the-diff bot commented Sep 12, 2024

PR Summary

  • Introduction of Location Column
    A new column titled 'location' has been added in the 'view-assets' section under 'account' to enhance data display.

  • Displaying Asset Location Names
    The 'location' attribute of an asset is now being utilized to show its name in the newly introduced column. This update provides an immediate glance at asset's location, improving user's understanding of the asset's status.

@snipe
Copy link
Owner

snipe commented Sep 16, 2024

Hi there - thanks for this! Can you please re-target this PR to point to the develop branch, per our Contributing documentation?

You don't need to close and re-open. After you create a pull request, you can modify the base branch so that the changes in the pull request are compared against a different branch. By changing the base branch of your original pull request rather than opening a new one with the correct base branch, you’ll be able to keep valuable work and discussion.

mhuxq

Thanks!

@snipe snipe changed the base branch from master to develop September 16, 2024 13:59
@snipe snipe changed the base branch from develop to master September 16, 2024 13:59
@snipe
Copy link
Owner

snipe commented Sep 16, 2024

(You might need to rebase your fork if you see a bunch of commits from me in your updated PR)

Copy link
Owner

@snipe snipe left a comment

Choose a reason for hiding this comment

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

Please retarget to develop - thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants