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

Lingo: Add painting display names #4707

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

Conversation

hatkirby
Copy link
Collaborator

@hatkirby hatkirby commented Mar 7, 2025

What is this fixing or adding?

This PR adds a human-friendly display name for every shuffleable painting in the game. This information can be used by the tracker to help players identify what paintings they are missing or want to go to. This new field is not added to the pickled datafile, and thus this change has no impact on generation whatsoever. However, the datafile did have to be regenerated because the hash of LL1.yaml changed.

Some paintings in crowded rooms were also reordered so that they show up in the tracker in clockwise order.

How was this tested?

Tested that the names displayed correctly with a beta version of the Lingo AP Tracker.

If this makes graphical changes, please attach screenshots.

@github-actions github-actions bot added the waiting-on: peer-review Issue/PR has not been reviewed by enough people yet. label Mar 7, 2025
@Exempt-Medic Exempt-Medic added the is: enhancement Issues requesting new features or pull requests implementing new features. label Mar 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
is: enhancement Issues requesting new features or pull requests implementing new features. waiting-on: peer-review Issue/PR has not been reviewed by enough people yet.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants