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

Feature cars contents #13

Merged
merged 4 commits into from
Nov 5, 2021
Merged

Feature cars contents #13

merged 4 commits into from
Nov 5, 2021

Conversation

Leif-The-Head
Copy link
Collaborator

New:

  • Wagon content type can be changed
    • if changing to another type, the car will be filled up
    • if changing to empty, the car will be emptied
  • Wagon content amount can be changed
    • only for cars that have a content type
  • Cars are distinguished by their type and each first line of number and name (this is to not clutter the list)
  • Inputs cannot exceed what is possible ingame

Not done in this version:

  • Shortcut buttons

Decided to not do before GUI:

  • Shortcuts for filling and emptying, as filling can be done by changing to another type then back and emptying by switching type to empty

Closes #8

Changes:

  • Moved everything rolling stock related except the menu itself into its own file
  • PEP compliance in some spots (but not all)

Leif-The-Head and others added 4 commits November 5, 2021 04:46
* content type can be changed
  * if changing to another type, the car will be filled up
  * if changing to empty, the car will be emptied
* content amount can be changed
  * only for cars that have a content type
* cars are distinguished by their type and each first line of number and name (this is to not clutter the list)
* inputs cannot exceed what is possible ingame

Not done in this version:
* shortcut buttons

Obsolete (as in I won't work on this):
* shortcuts for filling and emptying, as filling can be done by changing to another type then back and emptying by switching type to empty

Changes:
* moved everything rolling stock related except the menu itself into its own file
* content type can be changed
  * if changing to another type, the car will be filled up
  * if changing to empty, the car will be emptied
* content amount can be changed
  * only for cars that have a content type
* cars are distinguished by their type and each first line of number and name (this is to not clutter the list)
* inputs cannot exceed what is possible ingame

Not done in this version:
* shortcut buttons

Obsolete (as in I won't work on this):
* shortcuts for filling and emptying, as filling can be done by changing to another type then back and emptying by switching type to empty

Changes:
* moved everything rolling stock related except the menu itself into its own file
* content type can be changed
  * if changing to another type, the car will be filled up
  * if changing to empty, the car will be emptied
* content amount can be changed
  * only for cars that have a content type
* cars are distinguished by their type and each first line of number and name (this is to not clutter the list)
* inputs cannot exceed what is possible ingame

Not done in this version:
* shortcut buttons

Obsolete (as in I won't work on this):
* shortcuts for filling and emptying, as filling can be done by changing to another type then back and emptying by switching type to empty

Changes:
* moved everything rolling stock related except the menu itself into its own file
@Leif-The-Head Leif-The-Head added the Feature A new feature label Nov 5, 2021
@Leif-The-Head Leif-The-Head added this to the v0.3.7 milestone Nov 5, 2021
@Leif-The-Head Leif-The-Head added this to To do in RROSE - v0.3.7 via automation Nov 5, 2021
@Leif-The-Head Leif-The-Head linked an issue Nov 5, 2021 that may be closed by this pull request
@Leif-The-Head Leif-The-Head moved this from To do to Done in RROSE - v0.3.7 Nov 5, 2021
@Leif-The-Head Leif-The-Head moved this from Done to Pending Review/Testing in RROSE - v0.3.7 Nov 5, 2021
Copy link
Owner

@JennyKmu JennyKmu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perfect !
In the future, Readme.md will be updated only right before the merge back to main branch for release. Header update in the main.py (version, contributors...) as well.
Maybe split PEP conformity pull requests from feature requests (easier to review and track down modifications)

@JennyKmu JennyKmu merged commit eee5d51 into dev_v037 Nov 5, 2021
RROSE - v0.3.7 automation moved this from Pending Review/Testing to Done Nov 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature A new feature
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

Feature: cars content editor
2 participants