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

D2k Starport prototype #21423

Closed
wants to merge 3 commits into from
Closed

Conversation

Porenutak
Copy link
Contributor

@Porenutak Porenutak commented May 12, 2024

Successor of #21295

What this PR do:

  • OG fly path and behaviour for Frigate
  • Paying up front for Starport units
  • BulkProductionQueue allowing to order multiple units
  • ProductionStarport trait for delivering multiple units
  • OG palette widget behavior for BulkproductionQueue
  • OG refund behaviour (Fix resource to cash exploit in ProductionQueue)

What isnt part of this PR:

  • Stockpiles
  • dynamic prices

TODO:
Purchase button.
Right now you can purchase order by middle mouse click on any icon in BulkProductionQueue.

Before I dig into Purchase widget/logic I need some feedback from you for BulkProductionQueue, StarportProduction, Activities and palleteWidget. So there consensus that this is the way and will be merged in future.

Know issues:

  • Queue numbers and ordered numbers overlap Possible solution:
    o Render ordered number in right corner instead of left.
    o Disable queue number for BulkproductionQueue
  • Lots of debug messages and comments for easier testing. Will by removed later
  • Other changes for easier debugging: Delivery delay to 300 instead 1500, Selectable frigate Will by removed later.
2024-05-12.10-28-02.mp4
2024-05-12.10-28-42.mp4

fix resources to cash exploit
and option to block queue if player has not enought money
@Porenutak
Copy link
Contributor Author

related #7055

- OG fly path and Frigate behaviour
- BulkProductionQueue
- ProductionStarport trait
- OG palette widget behavior for Bulk queue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant