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

Add history tab to bounty console #2473

Merged
merged 7 commits into from
Dec 18, 2024

Conversation

BarryNorfolk
Copy link
Contributor

@BarryNorfolk BarryNorfolk commented Dec 18, 2024

About the PR

Added a history tab to the cargo bounty console.
DeltaV port of space-wizards/space-station-14#33932. Just cherry picked the changes across and added an extra commit for adding DeltaV comments literally everywhere.

Why / Balance

Knowing whether or not any bounties have been completed and when is important for people in charge of logistics, or command for that matter. It's also important to know who skipped what bounties so people can either teach which ones to not skip, or nod sagely at their good skip choices.

Technical details

New to the codebase and C# in general, but seemed somewhat straightforward to add.

Media

396848148-8bf389a5-3320-48b7-8739-498aef40b6d9

Requirements

  • I have tested all added content and changes.
  • I have added media to this PR or it does not require an ingame showcase.

Breaking changes

🆑

  • add: Added a history tab to the bounty computer, showing all past completed and skipped orders (and who skipped them).

@BarryNorfolk BarryNorfolk requested a review from a team as a code owner December 18, 2024 11:59
@github-actions github-actions bot added Changes: UI Changes: C# Changes any cs files Changes: Localization Changes any ftl files S: Needs Review size/M 64-255 lines labels Dec 18, 2024
@deltanedas
Copy link
Member

remove the deltav comments as it will just make merging upstream more annoying instead of just being skipped

@BarryNorfolk
Copy link
Contributor Author

remove the deltav comments as it will just make merging upstream more annoying instead of just being skipped

You want me to keep the files in DeltaV places or just remove the comments.

@deltanedas
Copy link
Member

just use the upstream commits as-is dont have comments or file changes (then thered be duplicate files when merging master, also bad)

@BarryNorfolk
Copy link
Contributor Author

Done.

@deltanedas deltanedas merged commit 600ef0e into DeltaV-Station:master Dec 18, 2024
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Changes any cs files Changes: Localization Changes any ftl files Changes: UI S: Needs Review size/M 64-255 lines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants