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.

Sign up for GitHub

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

Jump to bottom

Fixed when the mail has already cleared #2063

Merged
merged 6 commits into from
Apr 18, 2019
Merged

Fixed when the mail has already cleared #2063

merged 6 commits into from
Apr 18, 2019

Conversation

montlikadani
Copy link
Contributor

Fixed when the player typed the /mail clear command but did not delete anything because the mail was already deleted.

@montlikadani
Copy link
Contributor Author

Sorry, so far I did not know what the NPE is, but now I know.

@mdcfe mdcfe added the type: enhancement Features and feature requests. label Oct 28, 2018
mdcfe
mdcfe previously approved these changes Oct 28, 2018
View reviewed changes
@Ichbinjoe Ichbinjoe added the status: waiting on author Pull requests that require changes from the author in order to merge. label Apr 17, 2019
@mdcfe mdcfe dismissed their stale review April 17, 2019 11:04

Outdated

@Ichbinjoe Ichbinjoe merged commit 3f5f9cc into EssentialsX:2.x Apr 18, 2019
caojohnny pushed a commit to caojohnny/Essentials that referenced this pull request May 3, 2019
@montlikadani
Fixed when the mail has already cleared (EssentialsX#2063)
ad41c8b
@montlikadani montlikadani deleted the patch-26 branch January 5, 2020 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mdcfe mdcfe mdcfe approved these changes

@Ichbinjoe Ichbinjoe Ichbinjoe approved these changes

Assignees
No one assigned
Labels
status: waiting on author Pull requests that require changes from the author in order to merge. type: enhancement Features and feature requests.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@montlikadani @mdcfe @Ichbinjoe