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

Whitelistremoves now requires either ban or whitelist, not both #994

Merged

Conversation

NullWanderer
Copy link
Contributor

About the PR

Admins asked for it, and it looks to be a mistake in the code

Changelog
🆑
DELTAVADMIN:

  • fix: whitelistremove now requires either ban or whitelist, not both.

@github-actions github-actions bot added the Changes: C# Changes any cs files label Mar 21, 2024
@NullWanderer NullWanderer enabled auto-merge (squash) March 27, 2024 23:44
@NullWanderer NullWanderer merged commit a2a3bcb into DeltaV-Station:master Mar 27, 2024
11 checks passed
VMSolidus pushed a commit to Simple-Station/Einstein-Engines that referenced this pull request Apr 8, 2024
## Mirror of PR #994: [Whitelistremoves now requires either ban or
whitelist, not both](DeltaV-Station/Delta-v#994)
from <img src="https://avatars.githubusercontent.com/u/131613340?v=4"
alt="DeltaV-Station" width="22"/>
[DeltaV-Station](https://github.com/DeltaV-Station)/[Delta-v](https://github.com/DeltaV-Station/Delta-v)

<aside>PR opened by <img
src="https://avatars.githubusercontent.com/u/56081759?v=4"
width="16"/><a href="https://github.com/NullWanderer"> NullWanderer</a>
at 2024-03-21 23:16:47 UTC</aside>
<aside>PR merged by <img
src="https://avatars.githubusercontent.com/u/56081759?v=4"
width="16"/><a href="https://github.com/NullWanderer"> NullWanderer</a>
at 2024-03-27 23:53:10 UTC</aside>
<sup>

`a2a3bcbb869530aa1511af8339d8abc2cc84eba8`

</sup>

---

PR changed 0 files with 0 additions and 0 deletions.

The PR had the following labels:
- Changes: C#


---

<details open="true"><summary><h1>Original Body</h1></summary>

> ## About the PR
> Admins asked for it, and it looks to be a mistake in the code
> 
> **Changelog**
> :cl:
> DELTAVADMIN:
> - fix: whitelistremove now requires either ban or whitelist, not both.
> 
> 


</details>

Co-authored-by: Null <56081759+NullWanderer@users.noreply.github.com>
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant