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

Sector-wide alerts, Frontier-specific alert text #2474

Merged
merged 8 commits into from
Dec 21, 2024

Conversation

whatston3
Copy link
Contributor

@whatston3 whatston3 commented Nov 28, 2024

About the PR

Communications consoles now change a sector-wide alert level, which is properly displayed on PDAs and emergency lights on any available ship. Alert levels now display Frontier-specific text, with reference to the sector, crews, etc.

This could be merged with the NFSD SOP concept of heat (particular heat levels require active station alert levels), but this is left as a TODO.

Why / Balance

Been a long time coming.

Fixes #1139.

How to test

  1. Set a Red alert level on Frontier Outpost.
  2. Warp to Expeditionary Lodge. Neat, the lights are on.
  3. Wander outside, check your PDA. Wow, the sector's in Red alert.

Media

This photo shows a ghost in space, checking their PDA during a Violet alert. The emergency lights inside the Expeditionary Lodge are on and appropriately set to the alert level.
image

Requirements

Breaking changes

Changelog

🆑

  • tweak: Alert levels are now sector-wide, with appropriate announcements.

@dvir001 dvir001 self-requested a review December 6, 2024 19:09
@github-actions github-actions bot added the S: Needs Review This PR is awaiting reviews label Dec 6, 2024
@noldevin
Copy link

noldevin commented Dec 7, 2024

If this could be combined with or replaced by heat that would be fantastic. Right now heat is fully player based and the only way to find it is to ask. The alert system is the perfect way to actually implement heat as a game mechanic and would definitely help the NFSD stay at the right escalation level.

Copy link
Contributor

github-actions bot commented Dec 8, 2024

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

@github-actions github-actions bot added the S: Merge Conflict This PR has conflicts that prevent merging label Dec 8, 2024
@github-actions github-actions bot removed the S: Merge Conflict This PR has conflicts that prevent merging label Dec 9, 2024
@dvir001
Copy link
Contributor

dvir001 commented Dec 21, 2024

Cool

image

@dvir001 dvir001 merged commit 1ee2f68 into new-frontiers-14:master Dec 21, 2024
12 checks passed
FrontierATC added a commit that referenced this pull request Dec 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C# FTL S: Needs Review This PR is awaiting reviews YML
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Alert levels are not in sync on other ships
3 participants