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

Add registry builders for SoundEvent and JukeboxSong #11805

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

Machine-Maker
Copy link
Member

@Machine-Maker Machine-Maker commented Dec 24, 2024

Draft until we decided that Either<L, R> is the way to go for the fluent getter at JukeboxSongRegistryEntry#soundEvent Either it is

Tasks

Preview Give feedback
  1. scope: api type: feature

@kennytv kennytv added the type: feature Request for a new Feature. label Dec 24, 2024
@Machine-Maker Machine-Maker force-pushed the feature/JukeboxSong-registry-builder branch 3 times, most recently from d32f547 to d4e8d54 Compare January 5, 2025 20:41
@Machine-Maker Machine-Maker marked this pull request as ready for review January 6, 2025 03:25
@Machine-Maker Machine-Maker requested a review from a team as a code owner January 6, 2025 03:25
@Machine-Maker Machine-Maker force-pushed the feature/JukeboxSong-registry-builder branch from 84790d4 to 7b1f1ce Compare January 11, 2025 17:42
@Machine-Maker Machine-Maker force-pushed the feature/JukeboxSong-registry-builder branch from 7b1f1ce to f459030 Compare January 11, 2025 19:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: feature Request for a new Feature.
Projects
Status: Awaiting review
Development

Successfully merging this pull request may close these issues.

3 participants