Skip to content

New Salvage Ruins#36963

Closed
southbridge-fur wants to merge 1 commit intospace-wizards:masterfrom
southbridge-fur:new-salvage-ruins
Closed

New Salvage Ruins#36963
southbridge-fur wants to merge 1 commit intospace-wizards:masterfrom
southbridge-fur:new-salvage-ruins

Conversation

@southbridge-fur
Copy link
Copy Markdown
Contributor

@southbridge-fur southbridge-fur commented Apr 26, 2025

About the PR

New Salvage Ruins
Depends upon the following PRs

Why / Balance

More salvage ruins are always nice.

Technical details

Media

Requirements

Breaking changes

Changelog

no cl, us map makers don't deserve them

@github-actions github-actions Bot added S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. size/XL Denotes a PR that changes 5000+ lines. Changes: No C# Changes: Requires no C# knowledge to review or fix this item. Changes: Map Changes: Might require knowledge of mapping. labels Apr 26, 2025
@southbridge-fur southbridge-fur mentioned this pull request Apr 26, 2025
Closed
2 tasks
@Errant-4 Errant-4 added P3: Standard Priority: Default priority for repository items. T: New Feature Type: New feature or content, or extending existing content D2: Medium Difficulty: A good amount of codebase knowledge required. S: Needs Review Status: Requires additional reviews before being fully accepted. Not to be replaced by S: Approved. A: Mapping Area: Maps, or tools used for mapping A: Cargo/Salvage Area: Cargo department or Salvage. and removed S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. labels May 12, 2025
@SlamBamActionman
Copy link
Copy Markdown
Member

SlamBamActionman commented Apr 17, 2026

Hello! Salvage will be undergoing a major rework to its gameplay in the very near future, part of which involves an overhaul to how wrecks are spawned and generated via the Salvage magnet. This includes a stronger control over the contents and balance of wrecks, and what is mapped on them.

As such, I will be closing this PR for now. Once the rework has established its new spawn mechanics and balance requirements, feel free to revisit these wreck ideas in the new format.

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

Reviewers

No reviews

Assignees

No one assigned

Labels

A: Cargo/Salvage Area: Cargo department or Salvage. A: Mapping Area: Maps, or tools used for mapping Changes: Map Changes: Might require knowledge of mapping. Changes: No C# Changes: Requires no C# knowledge to review or fix this item. D2: Medium Difficulty: A good amount of codebase knowledge required. P3: Standard Priority: Default priority for repository items. S: Needs Review Status: Requires additional reviews before being fully accepted. Not to be replaced by S: Approved. size/XL Denotes a PR that changes 5000+ lines. T: New Feature Type: New feature or content, or extending existing content

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@southbridge-fur @SlamBamActionman @Errant-4