-
-
Notifications
You must be signed in to change notification settings - Fork 875
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
Fix #8153: Report autoreplace failure when new vehicle cannot carry the cargo #8169
Conversation
You should definitely try to include as much useful information to the player as possible:
|
81d6329
to
26e34e2
Compare
Hello @Chr12t0pher ; it has been a while since you opened this draft Pull Request. Is it ready for review? Or were you still working on this? :) If it was ready, please do rebase it and hit the Thank you! |
26e34e2
to
595afb4
Compare
595afb4
to
2c372d3
Compare
I rebased your work and made some minor coding-style fixes, nothing crazy :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
… no orders In GetIncompatibleRefitOrderIdForAutoreplace
Fixes #8153