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

I've covered wars, you know: Press camera drone & microphone changes #20289

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

FearTheGabby
Copy link
Contributor

image

See changelog. These changes are to hopefully both allow the Journalist to feel more fulfilled, especially during Odyssey rounds, as well as to let the Journalist have a one-on-one conversation broadcasted to the Entertainment channel without capturing everyone else's voice in range.

@FearTheGabby
Copy link
Contributor Author

!review

Comment on lines +148 to 150
if(!is_contact_area(get_area(C)) && !LAZYISIN(C.network, NETWORK_NEWS))
to_chat(user, SPAN_NOTICE("This camera is too far away to connect to!"))
return FALSE
Copy link
Contributor

Choose a reason for hiding this comment

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

does this respect overmap distance?
so you can't transmit from offship to horizon when they're on opposite ends of the overmap

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I don't believe so and I'm not sure how to go about implementing that

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