Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Old vehicles as a seperate news entry #2760

Closed
DorpsGek opened this issue Mar 23, 2009 · 10 comments
Closed

Old vehicles as a seperate news entry #2760

DorpsGek opened this issue Mar 23, 2009 · 10 comments
Labels
component: interface This is an interface issue flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) patch from FlySpray This issue is in fact a Patch, but imported from FlySrpay

Comments

@DorpsGek
Copy link
Member

KenjiE20 opened the ticket and wrote:

The attached patch splits lost trains from the 'Advice / Info on players vehicles' News entry. This is useful in games without breakdowns and non-expiring vehicles, so you don't loose a lost train entry amongst the old vehicle news.

Attachments

Reported version: trunk
Operating system: All


This issue was imported from FlySpray: https://bugs.openttd.org/task/2760
@DorpsGek
Copy link
Member Author

KenjiE20 wrote:

New version of patch, now separates vehicle age instead, so you can get all the other advice rather than just lost trains.

Attachments


This comment was imported from FlySpray: https://bugs.openttd.org/task/2760#comment5933

@DorpsGek
Copy link
Member Author

Alberth wrote:

Making a seperate category for a single news item doesn't sound like the best solution to me, it kills the whole category concept.


This comment was imported from FlySpray: https://bugs.openttd.org/task/2760#comment5936

@DorpsGek
Copy link
Member Author

KenjiE20 wrote:

The trouble is, if you run without breakdowns and auto-renew on, you can end up with a lot of old trains constantly popping up messages, making it much easier to miss other problems with vehicles (lost / void orders)
Mostly I did this to see if I could, and figured someone else might find it useful.

Updated against r16127

Attachments


This comment was imported from FlySpray: https://bugs.openttd.org/task/2760#comment5959

@DorpsGek
Copy link
Member Author

DorpsGek commented May 2, 2009

KenjiE20 wrote:

Updated against r16206

Attachments


This comment was imported from FlySpray: https://bugs.openttd.org/task/2760#comment6012

@DorpsGek
Copy link
Member Author

KenjiE20 wrote:

Updated against r16404

Attachments


This comment was imported from FlySpray: https://bugs.openttd.org/task/2760#comment6103

@DorpsGek
Copy link
Member Author

KenjiE20 wrote:

Updated against r16450

Attachments


This comment was imported from FlySpray: https://bugs.openttd.org/task/2760#comment6123

@DorpsGek
Copy link
Member Author

KenjiE20 wrote:

No changes made, but updated against 16782 and in a diff

Attachments


This comment was imported from FlySpray: https://bugs.openttd.org/task/2760#comment6339

@DorpsGek
Copy link
Member Author

KenjiE20 wrote:

Updated against r17224

Attachments


This comment was imported from FlySpray: https://bugs.openttd.org/task/2760#comment6514

@DorpsGek
Copy link
Member Author

DorpsGek commented Sep 7, 2009

KenjiE20 wrote:

Updated against r17425

Attachments


This comment was imported from FlySpray: https://bugs.openttd.org/task/2760#comment6599

@DorpsGek
Copy link
Member Author

andythenorth closed the ticket.

Reason for closing: Won't implement

Mass closure of patch tickets with no commentary for >5 years. Goal is to reduce patch queue as an experiment to see if it aids faster reviewing and rejection/acceptance (it may not). If this offends you and the patch is maintained and compiles with current trunk, discuss with andythenorth in irc. (andythenorth has no ability to review patches but can re-open tickets).


This comment was imported from FlySpray: https://bugs.openttd.org/task/2760

@DorpsGek DorpsGek added component: interface This is an interface issue flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) wontfix patch from FlySpray This issue is in fact a Patch, but imported from FlySrpay labels Apr 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: interface This is an interface issue flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) patch from FlySpray This issue is in fact a Patch, but imported from FlySrpay
Projects
None yet
Development

No branches or pull requests

1 participant