Skip to content

Don't pre-generate the pattern on wandering trader scrolls - #1363

Open
Robotgiggle wants to merge 5 commits into
mainfrom
trader-scroll-fix
Open

Robotgiggle wants to merge 5 commits into
mainfrom
trader-scroll-fix

Conversation

@Robotgiggle

@Robotgiggle Robotgiggle commented Nov 2, 2026 •

Copy link
Copy Markdown
Member
  • Fixes Pattern on wandering trader scrolls can be read via F3+I without purchasing them #1357.
  • Fixes Ancient Scrolls immediately refilling themselves when you try to erase them.
  • Abstracts the "get a list of ResourceKeys for all the great spells" behavior out of various places and into HexUtils to avoid reusing the exact same code in three different places.
  • Changes the item name for non-large ancient scrolls to the standard Ancient Scroll of <spell> rather than How did you get this item of <spell> since packmakers should be able to use other sizes if they want to.

@Robotgiggle Robotgiggle added enhancement New feature or request 1.20 labels Nov 2, 2026
@github-project-automation github-project-automation Bot moved this to 📋 Backlog in Hex Casting Nov 2, 2026
val perWorldKeys = mutableListOf<ResourceKey<ActionRegistryEntry>>()
val regi = IXplatAbstractions.INSTANCE.getActionRegistry()
for (key in regi.registryKeySet()) {
if (isOfTag(regi, key, HexTags.Actions.PER_WORLD_PATTERN)) {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is there not a better way to get all the entries of a tag??

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1.20 enhancement New feature or request

Projects

Status: 📋 Backlog

Development

Successfully merging this pull request may close these issues.

Pattern on wandering trader scrolls can be read via F3+I without purchasing them

2 participants