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.

Sign up for GitHub

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

Jump to bottom

Holiday Hoarders missing on h6 #504

Closed
RDIL opened this issue Aug 10, 2024 · 1 comment · Fixed by #550
Closed

Holiday Hoarders missing on h6 #504

RDIL opened this issue Aug 10, 2024 · 1 comment · Fixed by #550
Assignees
@AnthonyFuller
Labels
bug Something isn't working! ui Related to the game's menus.

Comments

@RDIL
Copy link
Member

RDIL commented Aug 10, 2024

Describe the bug
It's not visible on the Paris destination.
@RDIL RDIL added bug Something isn't working! ui Related to the game's menus. labels Aug 10, 2024
@AnthonyFuller
Copy link
Contributor

AnthonyFuller commented Aug 11, 2024 •
edited
Loading

Same issue as #505, it is because they changed the location data.

AnthonyFuller added a commit that referenced this issue Jan 4, 2025
@AnthonyFuller
refactor: game versioned missionsInLocations and fix certain contract…
3bfe2ba 
…s when resolved

Fixes #510 #505 #504
@AnthonyFuller AnthonyFuller mentioned this issue Jan 4, 2025
Merged
@AnthonyFuller AnthonyFuller linked a pull request Jan 4, 2025 that will close this issue
refactor!: game versioned missionsInLocation, fix certain contracts when resolved, rename missionsInLocations -> missionsInLocation #550
Merged
@RDIL RDIL closed this as completed in #550 Jan 15, 2025
RDIL added a commit that referenced this issue Jan 15, 2025
@RDIL
refactor!: game versioned missionsInLocation, fix certain contracts w…
17ba3ea 
…hen resolved, rename missionsInLocations -> missionsInLocation (#550)

Fixes #510 #505 #504

Needs testing and checking over more than twice.

## Breaking Changes
- `controller.missionsInLocations` is now
`controller.missionsInLocation` (as it always should have been!).
- The aforementioned `controller.missionsInLocations` is now **split by
game version**.
- `controller.addEscalation` has a new parameter: `(groupContract,
locationId, **gameVersion**, ...levels)` (highlighted using asterisks).

---

Also this removes some terrible hacks we've had to do. We should've
always done it like this!
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@AnthonyFuller AnthonyFuller

Labels
bug Something isn't working! ui Related to the game's menus.
Projects
None yet
Milestone
No milestone
2 participants
@AnthonyFuller @RDIL