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

Feature/track move flyblocks #700

Merged
merged 9 commits into from
Aug 25, 2024

Conversation

Intybyte
Copy link
Contributor

@Intybyte Intybyte commented Aug 24, 2024

Describe in detail what your pull request accomplishes

Add craft methods to get:
Move Block Amount
Flyblock Amount

Related issues:

#678

Checklist

  • Track Move blocks
  • Track Flyblocks
  • Combine the StatusSign calculation
  • Tested

@goodroach
Copy link
Contributor

goodroach commented Aug 24, 2024

Aren't there methods already in place that do that? Try checking the code for the status sign for an idea of how this works.

Edit: There isn't but it seems you made it easier to access. I'm not sure if this is the correct way to do it, considering that someone might add a different set of blocks required for the craft. I think these implementations should be up to the programmer making the add-on.

Copy link
Contributor

@TylerS1066 TylerS1066 left a comment

Choose a reason for hiding this comment

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

Looks good, just a few comments on clean up and style.

@TylerS1066
Copy link
Contributor

Tested and working well!

@TylerS1066 TylerS1066 linked an issue Aug 25, 2024 that may be closed by this pull request
@TylerS1066 TylerS1066 merged commit 71e87b2 into APDevTeam:main Aug 25, 2024
1 check passed
@Intybyte Intybyte deleted the feature/trackMoveFlyblocks branch August 25, 2024 19:16
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.

Add craft methods to get the block amount of a craft
3 participants