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

[Bug]: User are displayed only with first name and lower capital letters #1028

Closed
jurrejansen opened this issue Jun 6, 2024 · 6 comments · Fixed by #1049
Closed

[Bug]: User are displayed only with first name and lower capital letters #1028

jurrejansen opened this issue Jun 6, 2024 · 6 comments · Fixed by #1049
Assignees
Labels
bug Something isn't working Priority: High Issues that are with very high priority

Comments

@jurrejansen
Copy link

jurrejansen commented Jun 6, 2024

Contact Details

No response

What happened?

Custodian badges are not showing full name of user when asset status is In custody.

Screenshot 2024-06-06 at 15 22 34
Screenshot 2024-06-06 at 14 45 30
Screenshot 2024-06-06 at 14 44 51

What is the expected behaviour?

The users should be displayed with both First name and Last name

Version

Hosted (app.shelf.nu)

What browsers are you seeing the problem on?

Safari

Relevant log output

No response

@jurrejansen jurrejansen added the bug Something isn't working label Jun 6, 2024
@carlosvirreira
Copy link
Contributor

ty for reporting @jurrejansen

@DonKoko
Copy link
Contributor

DonKoko commented Jun 7, 2024

@jurrejansen I cannot re-create either the dropdown nor the table issue. Please invite me to the workspace where this happens so I can check it out in detail. I have a suspicion what the reason could be but I need more info so I can search for it in the DB.
Thanks for reporting.

@jurrejansen
Copy link
Author

Hi @DonKoko I am now experiencing blank menu items when checking out. Down below are 2 screenrecordings both recorded on the app.shelf. In the demo workspace it seems that the user that is appearing is blank is your user that was invited to investigate this issue.

https://github.com/Shelf-nu/shelf.nu/assets/76112272/ccea5e02-d872-43c3-a1ac-474e9061300f
https://github.com/Shelf-nu/shelf.nu/assets/76112272/4e07f38f-e9d6-433a-97de-33ac52ba7c33

Screenshot 2024-06-11 at 13 08 11
Screenshot 2024-06-11 at 13 08 27

@DonKoko DonKoko added the Priority: High Issues that are with very high priority label Jun 11, 2024
@DonKoko
Copy link
Contributor

DonKoko commented Jun 11, 2024

@jurrejansen @carlosvirreira released a fix to live with lots of improvements.
This issue opened a can of worms but ended up being amazing improvement. Please re-open if you notice any issues.

@DonKoko
Copy link
Contributor

DonKoko commented Jun 11, 2024

OKey we still have some places that need adjustment, namely Notes:
image
image

@DonKoko DonKoko reopened this Jun 11, 2024
@DonKoko
Copy link
Contributor

DonKoko commented Jun 12, 2024

@carlosvirreira this has been resolved. I had missed a few places. Also fixing this helped us find some other bugs related to NRM and bookings and custodians. Those are fixed as well.

@DonKoko DonKoko closed this as completed Jun 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Priority: High Issues that are with very high priority
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants