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

Vehicle Groups: improve error message when group hierarchy fails #5977

Closed
DorpsGek opened this issue Apr 11, 2014 · 1 comment
Closed

Vehicle Groups: improve error message when group hierarchy fails #5977

DorpsGek opened this issue Apr 11, 2014 · 1 comment
Assignees
Labels
bug Something isn't working component: interface This is an interface issue flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) good first issue Good for newcomers patch from FlySpray This issue is in fact a Patch, but imported from FlySrpay

Comments

@DorpsGek
Copy link
Member

juanjo opened the ticket and wrote:

In the recent feature about group hierarchy, the error message doesn't explain that loops in group hierarchy cannot be created.

I think the error message could explain why that action cannot be done, as some of the error messages do: "That action can't be done... Reason is that..."

Reported version: trunk
Operating system: All


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

3298 wrote:

I have a patch for this. Not much to say about it, it simply implements the proposal. I've used this myself for a year, so the patch includes the translation relevant for me. If that's not wanted, it should be trivial to remove that part.

Attachments


This comment was imported from FlySpray: https://bugs.openttd.org/task/5977#comment14540

@DorpsGek DorpsGek added Core flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) labels Apr 7, 2018
@andythenorth andythenorth added the component: interface This is an interface issue label Apr 13, 2018
@TrueBrain TrueBrain added patch from FlySpray This issue is in fact a Patch, but imported from FlySrpay bug Something isn't working good first issue Good for newcomers and removed enhancement from FlySpray labels Apr 13, 2018
@andythenorth andythenorth added the stale Stale issues label Jan 5, 2019
@LordAro LordAro self-assigned this Jan 12, 2019
@LordAro LordAro removed the stale Stale issues label Jan 12, 2019
LordAro added a commit to LordAro/OpenTTD that referenced this issue Mar 26, 2019
LordAro added a commit to LordAro/OpenTTD that referenced this issue Mar 26, 2019
LordAro added a commit to LordAro/OpenTTD that referenced this issue Mar 26, 2019
PeterN pushed a commit that referenced this issue Mar 27, 2019
douiwby pushed a commit to douiwby/OpenTTD that referenced this issue Apr 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working component: interface This is an interface issue flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) good first issue Good for newcomers patch from FlySpray This issue is in fact a Patch, but imported from FlySrpay
Projects
None yet
Development

No branches or pull requests

4 participants