nibuen / SpaceAlertMissionGenerator

Mission Generator For Space Alert Board Game On Android
https://blog.iterary.com/
MIT License
15 stars 9 forks source link

Prints a warning that the expansion is needed for double action missions #2

Closed tompen closed 10 years ago

tompen commented 10 years ago

Prints a warning in the mission log that double action cards from the expansion is required when selecting a Double action mission.

The warning is cleared once the mission starts.

tompen commented 10 years ago

Hi!

What do you think about this change? The screen will now look like this after you select a double action mission: screenshot_2014-07-31-00-10-14

I can probably get a german translation of the new string if you think the patch is good.

nibuen commented 10 years ago

Looks good, I wonder if it should be replaced once started though. So doesn't stay in mission log. I guess not big deal either way though. Please get german translation somehow and I will merge. (We can do a duolingo request if need be).

nibuen commented 10 years ago

oh sorry just noticed code looks like it already clears it. Great then yea just german translation would be great

tompen commented 10 years ago

Yes, the warning is cleared when the mission starts.

I will then go ahead and try to get a german translation of the new string.

roubert commented 10 years ago

For the German translation, I suggest the following:

Bitte beachten: Die Doppelaktionskarten aus der Erweiterung "Space Alert - Unendliche Weiten" ist erforderlich, um diese Mission zu spielen.

nibuen commented 10 years ago

@roubert Thanks for the support!

roubert commented 10 years ago

Oops, sorry, those cards should be plural:

Bitte beachten: Die Doppelaktionskarten aus der Erweiterung "Space Alert - Unendliche Weiten" sind erforderlich, um diese Mission zu spielen.

tompen commented 10 years ago

Thanks @roubert! I have now added your suggestion as the german translation.

nibuen commented 10 years ago

Will start pre-release build with plans on releasing in couple days.

nibuen commented 10 years ago

Sorry not sure why this doesn't show as your merge.