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

Robot Names Can No Longer Be 0 Characters Long #21907

Merged
merged 3 commits into from
Aug 16, 2023

Conversation

Daylight2
Copy link
Contributor

What Does This PR Do

Fixes #21841 - all robot names must now be at least 1 character long. Also fixed a bug that made the rename dialog pop up twice for floorbots.

Why It's Good For The Game

Things not having names bad.

Testing

Tried giving empty names to the most popular robots, tried giving a normal name to a robot.

Changelog

🆑
fix: Robot names can no longer be 0 characters long
/:cl:

@ParadiseSS13-Bot ParadiseSS13-Bot added the -Status: Awaiting type assignment This PR is waiting for its type to be assigned internally label Aug 3, 2023
Copy link
Contributor

@lewcc lewcc left a comment

Choose a reason for hiding this comment

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

Should be able to make this one change across all of your current changes.

@SteelSlayer SteelSlayer added the Fix This PR will fix an issue in the game label Aug 5, 2023
@ParadiseSS13-Bot ParadiseSS13-Bot added -Status: Awaiting review This PR is awaiting review from the review team and removed -Status: Awaiting type assignment This PR is waiting for its type to be assigned internally labels Aug 5, 2023
Copy link
Member

@S34NW S34NW left a comment

Choose a reason for hiding this comment

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

Annoying that there's a lot of copy paste here, but it's out of scope for you to tidy all of that up

@S34NW S34NW merged commit 1434695 into ParadiseSS13:master Aug 16, 2023
15 checks passed
Furrior pushed a commit to Furrior/Paradise-Remake that referenced this pull request Aug 18, 2023
* Add minimum length + warning

* Remove nullcheck

* The Minimalist Code Initiative
Furrior pushed a commit to ss220club/Paradise-SS220 that referenced this pull request Aug 18, 2023
* Add minimum length + warning

* Remove nullcheck

* The Minimalist Code Initiative
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-Status: Awaiting review This PR is awaiting review from the review team Fix This PR will fix an issue in the game
Projects
None yet
Development

Successfully merging this pull request may close these issues.

You can make bots with empty names.
6 participants