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

Fire an advancement trigger when reading a book #715

Open
wants to merge 1 commit into
base: 1.20.x
Choose a base branch
from

Conversation

TheRealWormbo
Copy link
Contributor

Trigger condition uses the book's resource location, and optionally a specific entry resource location and page number. However, the latter are only triggered by server code opening the book at that entry and page, not by the player going there via interactions with the opened book.

@TheRealWormbo TheRealWormbo marked this pull request as ready for review December 31, 2023 22:58
@TheRealWormbo
Copy link
Contributor Author

This PR has been tested using the following Botania code changes: TheRealWormbo/Botania@2c40fe8
The advancement for reading the Lexica Botania was successfully awarded on both Forge and Fabric when reading the book in item form, through a lectern, or via the /open-patchouli-book command.

@TheRealWormbo
Copy link
Contributor Author

The 1.20.1 version of this PR is archived at https://github.com/TheRealWormbo/Patchouli/tree/1.20.1-reading-advancement-trigger, in case that version of Patchouli still receives updates.

The following update to this PR is for 1.20.4, but lacking a corresponding Botania version, it could not be tested thoroughly.

@TheRealWormbo TheRealWormbo force-pushed the 1.20-reading-advancement-trigger branch from f45a1c2 to 1ba70b0 Compare April 20, 2024 18:53
@TheRealWormbo TheRealWormbo force-pushed the 1.20-reading-advancement-trigger branch from 1ba70b0 to 6bb02d5 Compare April 20, 2024 21:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant