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

[Bug] Closing the scenario Editor closes App (i.s.o. going to main menu) #1435

Closed
antontimmermans opened this issue Jan 15, 2021 · 0 comments
Assignees
Labels
bug To be tested Dev is done, it should now be tested

Comments

@antontimmermans
Copy link
Collaborator

antontimmermans commented Jan 15, 2021

Describe the bug

The valkyrie app exits when I leave the scenario test to go back to the editor.

About the scenario:

  • It can be played with Base set only. (required expansions: None)
  • SoA, PotS, BtT and RN (or MOM1E) give additional variation (monsters, conditions)
  • BtT and RN (or MOM1E) give additional layouts.

This issue only happens when not all expansions are selected. Note that the scenario verifies the # variables before using any of the related content.
The log file contains many unnecessary warnings like: Key not found: UIEpilogueContinueButton.uitext
Errors in the log file seem to point towards the missing BtT expansion: Error: Failed to located TileSide: 'TileSideEntryHallMAD23' in quest component: 'TileHallway1BtT'

  1. The steps I wrote below to reproduce do NOT call any event to load this tile.
  2. the errors seem to point only to the missing BtT and not to any other missing expansion.

Scenario

Possessed (new scenario - under development) available on request for test (PM me on discord)

Steps to reproduce the behavior:

Select Content -> Deselect all expansions
Scenario editor -> load the scenario in the editor
Hit Save & Test to start the scenario
Hit Menu -> Editor

Expected behavior

go back to main menu if you leave the editor with or without the required expansions.

LogFile

output_log.txt

Screenshots

Valkyrie Version

2.5.1

Desktop

  • OS: windows
  • Browser chrome
  • Version10

Additional context

@mayjak mayjak self-assigned this Jan 16, 2021
@mayjak mayjak added the To be tested Dev is done, it should now be tested label Jan 17, 2021
@mayjak mayjak closed this as completed in 8fab99a Feb 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug To be tested Dev is done, it should now be tested
Projects
None yet
Development

No branches or pull requests

2 participants