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

Adds support for slabs and stairs #485

Merged
merged 10 commits into from
Jan 28, 2025

Conversation

KairuDeibisu
Copy link
Contributor

@KairuDeibisu KairuDeibisu commented Jan 20, 2025

This merge adds support for slabs and stairs.

The framework was made by @Bafran and expanded to support stairs.

Levers can be added in a latter pull request. For now, I want to get in the basics.

Untitled.video.7.1.mp4

@KairuDeibisu KairuDeibisu changed the title Add properties support Add support for lever Jan 20, 2025
@Snowiiii
Copy link
Member

So whats next @KairuDeibisu. Please use a better PR description than this

@KairuDeibisu KairuDeibisu changed the title Add support for lever Making levers attached to blocks instead of defaulting to North. Jan 23, 2025
@KairuDeibisu KairuDeibisu changed the title Making levers attached to blocks instead of defaulting to North. Making levers attached to blocks instead of always North Jan 23, 2025
@KairuDeibisu KairuDeibisu changed the title Making levers attached to blocks instead of always North Levers should attach to blocks instead of facing north Jan 23, 2025
@KairuDeibisu KairuDeibisu changed the title Levers should attach to blocks instead of facing north Levers should attach to blocks Jan 23, 2025
@KairuDeibisu
Copy link
Contributor Author

So whats next @KairuDeibisu. Please use a better PR description than this

I've updated it.

@KairuDeibisu KairuDeibisu changed the title Levers should attach to blocks Adds support for slabs and stairs Jan 25, 2025
@KairuDeibisu KairuDeibisu marked this pull request as ready for review January 25, 2025 23:24
@Snowiiii
Copy link
Member

I wonder if this supports properties which are loaded directly from anvil, and also the other way. So is this saved properly ?

@@ -1 +1,9 @@
pub mod entity_registry;

#[derive(Debug, PartialEq, Clone, Copy)]
pub enum FacingDirection {
Copy link
Member

Choose a reason for hiding this comment

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

This is already defined in pumpkin-world/block/mod
as BlockFace. We may want to rename that to Direction in place it in pumpkin-util/math

@OfficialKris OfficialKris mentioned this pull request Jan 27, 2025
3 tasks
@Snowiiii
Copy link
Member

Overall this works fine already. So i think its worth merging and will help #490 and closes #447

@Snowiiii Snowiiii merged commit 6c4eb88 into Pumpkin-MC:master Jan 28, 2025
9 checks passed
@Snowiiii
Copy link
Member

Thank you @KairuDeibisu and @Bafran

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.

3 participants