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

[LoW] Inconsistency in Stated Scenario length between SP/MP #7218

Closed
knyghtmare opened this issue Dec 20, 2022 · 0 comments · Fixed by #7285
Closed

[LoW] Inconsistency in Stated Scenario length between SP/MP #7218

knyghtmare opened this issue Dec 20, 2022 · 0 comments · Fixed by #7285
Labels
Bug Issues involving unexpected behavior. Campaign (any) Deprecated tag, replaced with separate tags for each mainline campaign Good first issue Issues deemed adequate for contributors without prior experience to work on. Prose Issues with prose text for the game.

Comments

@knyghtmare
Copy link
Member

Scenario description says that the campaign is 18 scenarios. This is true...for single-player mode.

HOWEVER, yes, you all didn't notice this one...in MP mode (as done in c2c4d81), one particular scenario is not accessible (S16). Thus, the campaign length in MP mode is actually 17.
@knyghtmare knyghtmare added Campaign (any) Deprecated tag, replaced with separate tags for each mainline campaign Good first issue Issues deemed adequate for contributors without prior experience to work on. Prose Issues with prose text for the game. labels Dec 20, 2022
@Wedge009 Wedge009 added the Bug Issues involving unexpected behavior. label Dec 20, 2022
knyghtmare added a commit to knyghtmare/wesnoth that referenced this issue Jan 16, 2023
@knyghtmare
LoW: fix scenario count inconsistency
29aae21 
Closes wesnoth#7218
@knyghtmare knyghtmare mentioned this issue Jan 16, 2023
Merged
stevecotton pushed a commit that referenced this issue Jan 20, 2023
@knyghtmare @stevecotton
LoW: fix scenario count inconsistency
e04858f 
Closes #7218
Wedge009 pushed a commit that referenced this issue Jan 20, 2023
@knyghtmare @Wedge009
LoW: fix scenario count inconsistency
c72ca64 
Closes #7218

(cherry picked from commit e04858f)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug Issues involving unexpected behavior. Campaign (any) Deprecated tag, replaced with separate tags for each mainline campaign Good first issue Issues deemed adequate for contributors without prior experience to work on. Prose Issues with prose text for the game.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[LoW] fix scenario count inconsistency knyghtmare/wesnoth
2 participants
@knyghtmare @Wedge009