search

CitiesSkylinesMods / TMPE

Cities: Skylines Traffic Manager: President Edition
https://steamcommunity.com/sharedfiles/filedetails/?id=1637663252
MIT License
569 stars 85 forks source link

What's New panel - support for hotfix version #1566

Closed krzychu124 closed 2 years ago

krzychu124 commented 2 years ago

Describe the problem

Recently pushed What's New update #1563 broke the changelog parsing (I hotfixed the hotfix and uploaded working solution to the workshop 😅 ). Version accepts only up to 4 comma separated numbers and fails on reading other text characters.

Notes or questions?

Should be introduce TMPEVersion type in replacement of C# Version and implement additional parser + comparer? How hotfix releases should be displayed in the What's New panel? I think we should display hotfix(es) on top and then below the version that was hotfixed so the user will see all changes regards to specific version instead of changelog just from the hotfix(panel stops loading anything older than last shown version).

@aubergine10 ideas?

originalfoo commented 2 years ago

@aubergine10 ideas?

https://github.com/CitiesSkylinesMods/TMPE/pull/1563#issuecomment-1120444985