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

Stopped 'Slimefun' Infernal Dust dropping #87

Merged
merged 2 commits into from
Jul 5, 2021

Conversation

Sefiraat
Copy link
Member

@Sefiraat Sefiraat commented Jul 5, 2021

Description

Infernal Dust drops two different stack types, one from STB's Drop and another from the MobDrop recipe type in Slimefun. The Slimefun drop includes Slimefun data making the two items not stack. This also inflates the drop rate beyond what is intended within STB.

Changes

Created a new 'dummy' recipe type to look the same as Slimefun's MobDrop recipe type for display on the guide without triggering Slimefun's drops.

Related Issues

Resolves #74

Checklist

  • I have fully tested the proposed changes and promise that they will not break everything into chaos.
  • I followed the existing code standards and didn't mess up the formatting.
  • I did my best to add documentation to any public classes or methods I added.
  • I have added Nonnull and Nullable annotations to my methods to indicate their behaviour for null values

@Sefiraat Sefiraat requested a review from a team as a code owner July 5, 2021 08:46
@Sefiraat Sefiraat changed the title Added dummy version of MobDrop recipe type Stopped 'Slimefun' Infernal Dust dropping Jul 5, 2021
@TheBusyBiscuit TheBusyBiscuit merged commit 7e2e8e4 into Slimefun:master Jul 5, 2021
@Sefiraat Sefiraat deleted the infernal-dust-split-drops branch July 6, 2021 19:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Infernal dusts - double names
2 participants