[PATCH] Send a phase up notification to your teammates

[Stan]

Currently only Petra does this. It would be nice that your allies gets notified you phase up, especially since shared vision is now a tech and not automatic.

[elexis]

Backlogging due to lack of progress.

[Bichtiades]

now the simulation sends a notification with just "player - type - event" fields, and the gui creates the message. I d like to hear suggestions on how to translate the message's string though.

[elexis]

• Adding keywords in accordance with wiki:SubmittingPatches.
• When to receive notificaitons IMO:
  • the player reached age 2 / 3
  • a player researched the wonder maxpop tech
  • the player built, captured or lost a civic center, fortress or wonder (these notifications would also be used in #3743)
  • I'm not convinced it is useful for markets, maybe only for the first market built?
• Send the notification from OnResearchFinished and OnGlobalOwnershipChanged in Player.js (since the player can cancel the research)
• Don't use "/allies", since
  • it can't reach observers
  • it means the text will be "Player1: I reached town phase", but it should be "Player1 has reached town phase", just like the defeat-message for example.
  • Instead, add a new type to g_FormatChatMessage which uses a new function formatTeamNotification in messages.js (similar to the other format***Message).
  • You can use g_Players[player].IsMutualAlly[g_ViewedPlayer] or something like that
• Options:
  • revert options.xml and make this a "general" option (unless you can prove that there will definitely be more options added in the near future)
• AI notifications:
  • The petra AI sends the same notifications.
  • Hence the duplicate code must be removed from headquarters.js and chatHelper.js (i.e. chatNewPhase and the calls to it and the then unused variables phaseName)
• Add yourself to the contributor list in your patch, see programming.json (either with or without realname)

[Itms]

I'm pushing that because the release is getting close :)

[Lionkanzen]

With a sound can be nice. An additive alert for messages I propose several in a topic.

[elexis]

Any updates? (I saw your working copy works again). There are several players who are excited about this feature to be included in alpha 21 and once done, you and your allies will notice the addition to the game in every match. It will shape the strategy discussions in the team chat :)

[elexis]

Backlogging due to lack of progress.

[elexis]

Patch at ​https://code.wildfiregames.com/D354

[elexis]

In 19445:

Display a chat notification if an ally has reached a new phase, optionally also if started or aborted.
Remove the equivalent AI chat notifications.

Differential Revision: ​https://code.wildfiregames.com/D354
Patch By: Polakrity
Fixes #3512