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

Departmental Intercoms #11422

Closed

Conversation

MarkusLarsson421
Copy link
Contributor

@MarkusLarsson421 MarkusLarsson421 commented Sep 2, 2024

About The Pull Request

Adds department specific and non-station intercoms. This includes NO mapping changes. Can be used whilst telecoms is down but can not have its frequency changed.

Why It's Good For The Game

Allows people to talk over coms for their given department even if telecoms is down.

Testing Photographs and Procedure

Screenshots&Videos

image

image

image

image

Changelog

🆑
add: Adds department specific intercoms (including non-station intercoms).
/:cl:

@kiekatz
Copy link
Contributor

kiekatz commented Sep 3, 2024

breaking into a secure frequency has never been easier.
now just as easy as getting into the department and stealing an intercom

@MarkusLarsson421
Copy link
Contributor Author

Yep! But you have to stay around it, which tends to be harder. It'll let you taunt Security easily or even calling for security and similarly. It also allows people who are not on the Medbay comms to ask for help in Medbay via their comms. Instead of yelling "MEDICAL DOCTOR TO MEDBAY!" over common comms.

@EvilDragonfiend
Copy link
Member

EvilDragonfiend commented Sep 6, 2024

This removes AI's ability to eavesdrop crews converstaion through adjusting intercom's channel to AI private.
I personally prefer more elegant way to control intercoms. maybe some advanced intercom stuff? something like that. we prolly need to discuss about tihs.

@Nakalon
Copy link
Contributor

Nakalon commented Sep 15, 2024

I like this idea but agree with what was said. No one uses intercoms as it is now except when power goes out

@MarkusLarsson421
Copy link
Contributor Author

I like this idea but agree with what was said. No one uses intercoms as it is now except when power goes out

If you were in Medbay or Security lobby and you needed their help but you don't have Security coms, you can now do that. Instead of yelling out onto common coms "MEDIC TO MEDBAY!!!". Also, it allows for the department to talk to each other when coms is out. Say you're a Doctor and you need something from Chemistry, now you'd be able to ask for it. Assuming they have an intercom nearby. It's not a major change, but it is one that I think will benefit the game.

Copy link

This pull request has conflicts, please resolve those before we can evaluate the pull request.

Copy link

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@MarkusLarsson421 MarkusLarsson421 marked this pull request as ready for review October 3, 2024 20:04
@MarkusLarsson421
Copy link
Contributor Author

MarkusLarsson421 commented Oct 3, 2024

This is ready for reviewing, didn't change much in the time but I did decide to not do any mapping changes at all. I will leave that for future mappers to do as they wish.

Changed this to be Add instead of a Balance change as it doesn't change anything as of now. Let me know if this should be reverted.

Copy link
Member

@EvilDragonfiend EvilDragonfiend left a comment

Choose a reason for hiding this comment

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

I don't like that it's not possible to rebuild one.
I think there should be a kind of minour version of radio key that is only allowed to go into intercom.
or /area should have a radio channel freq, and it should enable the department frequency.

@EvilDragonfiend
Copy link
Member

It seems you're not sure what to do. Try this approach

/obj/item/radio/intercom/recalculateChannels()
	..()
	var/area/intercom_area = get_area(src)
	if(!intercom_area)
		return
	
	for(var/each_channel in intercom_area.intercom_radio_channels)
		channels[each_channel] = TRUE

/area
	var/intercom_radio_channels

/area/chief_medical_officer_room
	intercom_radio_channels = list(RADIO_CHANNEL_COMMAND, RADIO_CHANNEL_MEDICAL)

@Rukofamicom
Copy link
Contributor

Due to personal circumstances, author has requested that all PRs be treated "as-is" for the time being.
Closing this one due to outstanding changes requested

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

Successfully merging this pull request may close these issues.

5 participants