* jaws of death
* I hate YAML
* open the gate
* I forgot about this one
* I forgor
* Fix croissant
* this didn't work actually rip
* made lights look better and undo on the boxing gloves
* small change
* Update Resources/Prototypes/Entities/Clothing/Hands/gloves.yml
Co-authored-by: IProduceWidgets <107586145+IProduceWidgets@users.noreply.github.com>
* baguette contraband, eat your evidence.
* suffix
---------
Co-authored-by: Princess Cheeseballs <66055347+Pronana@users.noreply.github.com>
Co-authored-by: IProduceWidgets <107586145+IProduceWidgets@users.noreply.github.com>
Fix PostgreSQL migration for ban DB refactor (#42609)
This didn't come up in testing. The ban hit foreign key was added before the data was migrated, so it didn't work.
Fix that.
Ban database refactor (#42495)
* Ban DB refactor seems to work at a basic level for PostgreSQL
* New ban creation API
Supports all the new functionality (multiple players/addresses/hwids/roles/rounds per ban).
* Make the migration irreversible
* Re-implement ban notifications
The server ID check is no longer done as admins may want to place bans spanning multiple rounds irrelevant of the source server.
* Fix some split query warnings
* Implement migration on SQLite
* More comments
* Remove required from ban reason
SS14.Admin changes would like this
* More missing AsSplitQuery() calls
* Fix missing ban type filter
* Fix old CreateServerBan API with permanent time
* Fix department and role ban commands with permanent time
* Re-add banhits navigation property
Dropped this on accident, SS14.Admin needs it.
* More ban API fixes.
* Don't fetch ban exemption info for role bans
Not relevant, reduces query performance
* Regenerate migrations
* Fix adminnotes command for players that never connected
Would blow up handling null player records. Not a new bug introduced by the refactor, but I ran into it.
* Great shame... I accidentally committed submodule update...
* Update GDPR scripts
* Fix sandbox violation
* Fix bans with duplicate info causing DB exceptions
Most notably happened with role bans, as multiple departments may include the same role.
Ban database refactor (#42495)
* Ban DB refactor seems to work at a basic level for PostgreSQL
* New ban creation API
Supports all the new functionality (multiple players/addresses/hwids/roles/rounds per ban).
* Make the migration irreversible
* Re-implement ban notifications
The server ID check is no longer done as admins may want to place bans spanning multiple rounds irrelevant of the source server.
* Fix some split query warnings
* Implement migration on SQLite
* More comments
* Remove required from ban reason
SS14.Admin changes would like this
* More missing AsSplitQuery() calls
* Fix missing ban type filter
* Fix old CreateServerBan API with permanent time
* Fix department and role ban commands with permanent time
* Re-add banhits navigation property
Dropped this on accident, SS14.Admin needs it.
* More ban API fixes.
* Don't fetch ban exemption info for role bans
Not relevant, reduces query performance
* Regenerate migrations
* Fix adminnotes command for players that never connected
Would blow up handling null player records. Not a new bug introduced by the refactor, but I ran into it.
* Great shame... I accidentally committed submodule update...
* Update GDPR scripts
* Fix sandbox violation
* Fix bans with duplicate info causing DB exceptions
Most notably happened with role bans, as multiple departments may include the same role.
* Ban DB refactor seems to work at a basic level for PostgreSQL
* New ban creation API
Supports all the new functionality (multiple players/addresses/hwids/roles/rounds per ban).
* Make the migration irreversible
* Re-implement ban notifications
The server ID check is no longer done as admins may want to place bans spanning multiple rounds irrelevant of the source server.
* Fix some split query warnings
* Implement migration on SQLite
* More comments
* Remove required from ban reason
SS14.Admin changes would like this
* More missing AsSplitQuery() calls
* Fix missing ban type filter
* Fix old CreateServerBan API with permanent time
* Fix department and role ban commands with permanent time
* Re-add banhits navigation property
Dropped this on accident, SS14.Admin needs it.
* More ban API fixes.
* Don't fetch ban exemption info for role bans
Not relevant, reduces query performance
* Regenerate migrations
* Fix adminnotes command for players that never connected
Would blow up handling null player records. Not a new bug introduced by the refactor, but I ran into it.
* Great shame... I accidentally committed submodule update...
* Update GDPR scripts
* Fix sandbox violation
* Fix bans with duplicate info causing DB exceptions
Most notably happened with role bans, as multiple departments may include the same role.
* Commit
* add the form post
* dv
* fixes
* Change wording
* Address review
* wording change
* Added some stuff
* New format
* bruh
* thanks perry!
* yes
* More fixes!
* typo
* Add a command to show the list, improve the UI slightly, split up command names
* Fix UI controller
* Add better comment
* Get rid of weird recursive thing
* Cleanup
* Work on moving feedback popups out of simulation
* Move round end screen subscription to feedback ui controller
* Finish moving feedback popups out of simulation
* Fix _ as parameter
* Clean up FeedbackPopupUIController
* Clean up commands
* Fix prototype yaml
* Fix openfeedbackpopup command description
* Update Resources/Locale/en-US/feedbackpopup/feedbackpopup.ftl
Co-authored-by: Simon <63975668+Simyon264@users.noreply.github.com>
* Apply suggestions from code review
Co-authored-by: Simon <63975668+Simyon264@users.noreply.github.com>
* Address reviews
* Address reviews
* Fix FeedbackPopupPrototype.cs using empty string instead of string.empty
* Address some more of the reviews, style nano is still trolling sadly
* Fix feedback popup styling
* Fix PopupPrototype ID field not having a setter
* Address reviews
* Add label when no feedback entries are present
Change link button to not show when no link is set
* Fix typo in feedback prototype
---------
Co-authored-by: beck-thompson <beck314159@hotmail.com>
Co-authored-by: SlamBamActionman <slambamactionman@gmail.com>
Co-authored-by: Simon <63975668+Simyon264@users.noreply.github.com>
* Commit
* add the form post
* dv
* fixes
* Change wording
* Address review
* wording change
* Added some stuff
* New format
* bruh
* thanks perry!
* yes
* More fixes!
* typo
* Add a command to show the list, improve the UI slightly, split up command names
* Fix UI controller
* Add better comment
* Get rid of weird recursive thing
* Cleanup
* Work on moving feedback popups out of simulation
* Move round end screen subscription to feedback ui controller
* Finish moving feedback popups out of simulation
* Fix _ as parameter
* Clean up FeedbackPopupUIController
* Clean up commands
* Fix prototype yaml
* Fix openfeedbackpopup command description
* Update Resources/Locale/en-US/feedbackpopup/feedbackpopup.ftl
Co-authored-by: Simon <63975668+Simyon264@users.noreply.github.com>
* Apply suggestions from code review
Co-authored-by: Simon <63975668+Simyon264@users.noreply.github.com>
* Address reviews
* Address reviews
* Fix FeedbackPopupPrototype.cs using empty string instead of string.empty
* Address some more of the reviews, style nano is still trolling sadly
* Fix feedback popup styling
* Fix PopupPrototype ID field not having a setter
* Address reviews
* Add label when no feedback entries are present
Change link button to not show when no link is set
---------
Co-authored-by: beck-thompson <beck314159@hotmail.com>
Co-authored-by: SlamBamActionman <slambamactionman@gmail.com>
Co-authored-by: Simon <63975668+Simyon264@users.noreply.github.com>
* Grappling rework - Grappling hooks are now physics-based
* still have no idea wtf is going on with portals but fixed a few bugs + cleanup
* bonus fixes + prep for optional-but-recommended engine PR
* dropkicking a stray comment outta here
* makes the impulses actually take into account the fucking relays, makes reeling cancel if the rope's already too short, and tweaks values
* reviews + cleanup + makes ungrapple behavior a bit more consistent
joint removal was removed from ungrapple because it mispredicts either way, and breaks grappling hooks attached to the grappling gun (always good to leave possibilities like that open)
* adds a hack to work around grids not caring about waking cross-grid joints
* makes use of dirtyfield(), defenestrates magic number
* y'know it'd probably be better if we were like actually awake before we made commits
* null-coalesce instead of if statement
* two changes
* dont datafield and fix up for sound overrides
---------
Co-authored-by: Princess Cheeseballs <66055347+Pronana@users.noreply.github.com>
* Simplify hands UI code
* i remembered about SortedHands in the component
* minor cleanup
---------
Co-authored-by: Princess Cheeseballs <66055347+Pronana@users.noreply.github.com>
