Skip to content

Conversation

@SomeTroglodyte
Copy link
Collaborator

@yairm210 , @RobLoach - my brain hurts and I feel stuck, I'm going to pause for a while. This is the last state of my approach to solve #14056

  • How verbose do we want to be - skip some transitions silently?
  • Would a common translation template covering several cases make sense?
  • Harbor: A connection only via Harbor counts as Road until Railroads are researched, then it counts as railroad for stats, right? Notifications would probably only care about the connection quality (which could be a function close to the bestRoadStatus one)? Or would someone appreciate the extreme detail to notify distinctly when a connection needs harbors and roads so it's susceptible to both naval blockade and pillagers?
  • Harbor "quality" (dep on railroads researched) is now lost once a CapitalConnectionsFinder(civInfo).find() is through, but could be preserved by using more enum entries?
  • Uh, what would be the effects of having cities with Harbors but not having researched The Wheel (may require mods, forgot the vanilla tech tree dependencies)?

Take your time, I'm trying for a Chaotic Evil TNO.

@github-actions
Copy link

github-actions bot commented Dec 1, 2025

This pull request has conflicts, please resolve those before we can evaluate the pull request.

# Conflicts:
#	core/src/com/unciv/logic/civilization/transients/CivInfoTransientCache.kt
@github-actions
Copy link

github-actions bot commented Dec 2, 2025

Conflicts have been resolved.

@github-actions github-actions bot removed the Conflicts label Dec 2, 2025
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.

1 participant