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

Paramedics can't do their job now. #67498

Closed
redguy999 opened this issue Jun 3, 2022 · 6 comments · Fixed by #67512
Closed

Paramedics can't do their job now. #67498

redguy999 opened this issue Jun 3, 2022 · 6 comments · Fixed by #67512
Labels
Not a bug This should just be a list of closed issue reports Oversight We have no quality control

Comments

@redguy999
Copy link
Contributor

redguy999 commented Jun 3, 2022

Reporting client version: 514.1584

Round ID:

184238 (It has been around for at least a few rounds now)

Testmerges:

Reproduction:

  1. Be a paramedic
  2. Attempt to enter any department that isn't medical
  3. notice you can't enter.

I don't see a PR removing paramedic's access to any of these departments (sans EVA), so I'm presuming this is a bug.

I can get removing Engineering access (though I don't like it), I can also understand removing cargo access, removing EVA is annoying, but science access basically only gave you access to nanites, and service basically only gave you access to the lathe, which doesn't mean much.

@Jolly-66
Copy link
Contributor

Jolly-66 commented Jun 3, 2022

Probably fallout from when Son of Space did access revamps and stuff

Probably not

@Son-of-Space
Copy link
Contributor

image

@Palpetinus
Copy link

This was last changed right here
#64969
Three months ago
Personal opinion: kinda killed paramedic for me. I always played the ol' search and rescue paramed, keeping the crew monitor on another screen, pinpointer (now LifeLine) open and in a pocket (now ID slot) and so on.
Not being able to go into engineering to claw the SM-Rads infested engineers out of there and similar such things, kinda ruined it for me. Oh well.

@JohnFulpWillard
Copy link
Member

JohnFulpWillard commented Jun 3, 2022

@Palpetinus
That PR is not the fault of it
The new ID changes reworked what has access to what, the one @Son-of-Space is saying isn't the cause.

Following screenshots are from Deltastation, before and after the refactor

Cargo
Mining access was enough to enter Cargo bay, that is not the case anymore. (Shaft Miners were instead given Cargo shuttle access so they aren't affected, which is a problem too because they're only supposed to have it on lowpop).
Old:
image
New:
image
image

Science
Paramedics have Research access, but not Science access, which is what was needed for the doors now.
Old:
image
image
New:
image
image

Engineering
They have Construction, not Engineering. In the past Construction was needed to enter Engineering front doors (it also removed Command's access to these areas at the same time).
Old:
image
image
image
New:
image
image

TL;DR Don't blame my nerf, it wasn't me.

@Son-of-Space
Copy link
Contributor

With John's comment, it makes the issue a little more complicated. General access is there to be the generalized access for members of that department. In order to make it so paramedics can have access to the front doors on a department, we would need to add a snowflake access specifically for paramedics and add it on the front doors of every department across all stations. That seems to me like it wouldn't be ideal especially since part of what I've been trying to do with the ID reworks is get to a place where we can SIMPLIFY our accesses rather than make them more complicated. We can't just give paramedics general access to all of the departments, either, for the same reasons that John put out in the PR you referenced. I will have to talk more with him on it and see if we can come up with a satisfying solution.

@Dorsisdwarf Dorsisdwarf added Not a bug This should just be a list of closed issue reports Oversight We have no quality control labels Jun 3, 2022
@Mothblocks
Copy link
Member

@Dorsisdwarf Which is it? "Not a bug" should only be closed issues, if this is unintentional then please remove it and just keep oversight

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Not a bug This should just be a list of closed issue reports Oversight We have no quality control
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants