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

Security records photos of crewmembers is broken #8328

Open
Fronsis opened this issue Jan 17, 2023 · 9 comments
Open

Security records photos of crewmembers is broken #8328

Fronsis opened this issue Jan 17, 2023 · 9 comments
Labels
Triaged Issues that are exempt from the stale action

Comments

@Fronsis
Copy link

Fronsis commented Jan 17, 2023

Title. Basically you can't see the pictures that are attached to the crewmembers on their security record, it's a shame because it can help identify people that are in need of a search/arrest/etc and knowing how they look it's a good feature, it has been broken for a couple of weeks i believe
Screenshot_2

@itsmeow
Copy link
Member

itsmeow commented Jan 22, 2023

I think you just need to reload the record. It's because the page loaded before the photo finished being sent to the client

@Fronsis
Copy link
Author

Fronsis commented Jan 22, 2023

I think you just need to reload the record. It's because the page loaded before the photo finished being sent to the client

Hmm, how do you reload the record? I've tried opening and closing it a couple of times(when i have to update the crime of someone, etc) and this seems to happen every round, at least on my end.

@Fronsis
Copy link
Author

Fronsis commented Feb 9, 2023

Still broken.

@Penwin0
Copy link
Member

Penwin0 commented Feb 23, 2023

The PR which mentioned this issue has nothing to do with it

@Fronsis
Copy link
Author

Fronsis commented Mar 11, 2023

Indeed, it was just as a bump to the issue and a hopeful comment, but yeah unfortunately's still present i've yet to manage to get the pictures fixed, and i think newscatter has a very similar issue with photos but i'll do another one for that since it's different

@MrEloda
Copy link
Contributor

MrEloda commented Apr 3, 2023

This issue isn't exclusive to the security records. It's the same when trying to post a picture on the newscaster.

@github-actions
Copy link

github-actions bot commented May 4, 2023

This issue has been inactive for long enough to be automatically marked as stale. If this was a bug report and hasn't been addressed yet, and is still a problem, please don't hesitate to notify a maintainer.

@github-actions github-actions bot added the Stale label May 4, 2023
@Fronsis
Copy link
Author

Fronsis commented Jul 13, 2023

Well, i'd like to uh.. notify that this is still an issue. @itsmeow i don't know what you mean by reload the records but i've tried everything and recently we were in need of using that to identify a person with X color of hair but we weren't able too and we had to do it the long way (print a wanted person of like 10 different people until we got the person) just to see the pic

Btw, since it's related weren't you working at one point at some sort of tgui upgraded version of this? i remember glimpses of screenshot of that

@Penwin0 Penwin0 added Triaged Issues that are exempt from the stale action and removed Stale labels Jul 13, 2023
@Tsar-Salat
Copy link
Contributor

Likely fixed by full replacement in #11545

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Triaged Issues that are exempt from the stale action
Projects
None yet
Development

No branches or pull requests

5 participants