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.

Sign up for GitHub

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

Jump to bottom

Always show item's status value on staff item screen #1742

Closed
jim opened this issue Nov 9, 2024 · 0 comments
Closed

Always show item's status value on staff item screen #1742

jim opened this issue Nov 9, 2024 · 0 comments
Assignees
@crismali

Comments

@jim
Copy link
Member

jim commented Nov 9, 2024

We currently reuse the member-facing code to display item status to staff. This obscures the actual value of status for that item and makes more work.

  1. Update admin item pages to always show the value of status.
  2. Also show "checked out", "overdue", etc., but after the other status value. So in some cases, we'll show two things.

Image
@jim jim converted this from a draft issue Nov 9, 2024
@crismali crismali self-assigned this Dec 17, 2024
@crismali crismali moved this from 📋 Backlog to 🏗 In progress in Circulate Development Dec 17, 2024
@crismali crismali mentioned this issue Dec 17, 2024
Merged
@crismali crismali moved this from 🏗 In progress to 👀 In review in Circulate Development Dec 17, 2024
jim pushed a commit that referenced this issue Dec 18, 2024
@crismali
Show the raw value of each item's status in addition to the member fa…
951901a 
…cing version in the admin interface (#1792)

# What it does

Displays `item.status` on the admin item's index and show pages in
addition to the `item_status_label` version.

# Why it is important

#1742 

# UI Change Screenshot

Admin items  index page:
![Screenshot 2024-12-17 at 4 33
21 PM](https://github.com/user-attachments/assets/9ee6750f-15e2-4927-bacc-31a46d572d22)

Admin item show page:
![Screenshot 2024-12-17 at 4 33
44 PM](https://github.com/user-attachments/assets/e3efb652-a48d-4067-823b-7068309e7a4c)

# Implementation notes

Just added the capitalized version of the status to both pages. Right
now it's background is always gray, but we could add some colors to make
it a little nicer if it's not too noisy.
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Merged in Circulate Development Dec 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@crismali crismali

Labels
None yet
Projects
Circulate Development
Status: ✅ Merged
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jim @crismali