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

Fix records consoles #1076

Merged
merged 2 commits into from
Mar 4, 2024
Merged

Fix records consoles #1076

merged 2 commits into from
Mar 4, 2024

Conversation

Cheackraze
Copy link
Member

About the PR

Makes it so SR and Sheriff can once again hire people

Why / Balance

Technical details

This will cause duplicate records entries when buying/selling ships, though it should be a minor thing, and maybe even a good thing so people cant hide their records like they used to

Media

  • I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

@github-actions github-actions bot added the Merge Conflict This PR has conflicts that prevent merging label Mar 4, 2024
Copy link
Contributor

github-actions bot commented Mar 4, 2024

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

@github-actions github-actions bot added the C# label Mar 4, 2024
# Conflicts:
#	Content.Server/Shipyard/Systems/ShipyardSystem.Consoles.cs
@github-actions github-actions bot removed the Merge Conflict This PR has conflicts that prevent merging label Mar 4, 2024
@dvir001 dvir001 requested a review from GreaseMonk March 4, 2024 20:11
@github-actions github-actions bot added the Status: Needs Review This PR is awaiting reviews label Mar 4, 2024
@dvir001 dvir001 self-requested a review March 4, 2024 20:11
@dvir001 dvir001 merged commit 4188a88 into master Mar 4, 2024
12 checks passed
@GreaseMonk GreaseMonk deleted the records-console-fixes branch March 5, 2024 09:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C# Status: Needs Review This PR is awaiting reviews
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants