Skip to content

Feature/alliance renewal#1

Merged
1brucben merged 3 commits into
1brucben:mainfrom
El-Magico777:feature/alliance-renewal
Jul 3, 2025
Merged

Feature/alliance renewal#1
1brucben merged 3 commits into
1brucben:mainfrom
El-Magico777:feature/alliance-renewal

Conversation

@El-Magico777
Copy link
Copy Markdown
Collaborator

@El-Magico777 El-Magico777 commented Jun 29, 2025

Description:

  1. Adds alliance renewal prompt 30 seconds before expiration,

image
image

  1. and an extra renewal prompt if the alliance has already expired. (from original PR but still useful, can be removed if needed)

image
image

This (1) helps prevent accidental attacks between allies when a renewal opportunity is missed
and (2) is still usefull when you missed the pre-expiration promt.

  • I have added screenshots for all UI updates
  • I confirm I have thoroughly tested these changes and take full responsibility for any bugs introduced
  • I understand that submitting code with bugs that could have been caught through manual testing blocks releases and new features for all contributors
  • I process any text displayed to the user through translateText() and I've added it to the en.json file
  • I have added relevant tests to the test directory

Discord username:

el_magico777

My 2nd ever PR and its a port from the first. Appreciate all tips to improve my work!

1brucben and others added 3 commits June 28, 2025 14:30
reduce productivity when nuked

update news modal

adjust investment

adjustproductivity

balance

fix productivity growth

adjust again

nerf investment

reduce default gold
…mpts, translations, and tests

- Implements alliance renewal and expiry functionality
- Shows renewal prompt only for expiring alliances involving local player
- Adds translations for all new alliance events (en/nl)
- Updates EventsDisplay for correct prompts and messages
- Adds AllianceExpireCheckExecution & AllianceExtensionExecution files
- Adds unit test for AllianceExtensionExecution
@El-Magico777 El-Magico777 marked this pull request as ready for review June 29, 2025 22:00
@1brucben 1brucben merged commit 87ec717 into 1brucben:main Jul 3, 2025
6 of 8 checks passed
@El-Magico777 El-Magico777 added this to the v0.1.11 milestone Feb 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants