Skip to content

look at my code B+) #4

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

Open
wants to merge 104 commits into
base: develop
Choose a base branch
from
Open

look at my code B+) #4

wants to merge 104 commits into from

Conversation

tangirene
Copy link
Contributor

ummmm there might be some merge conflicts cause i added a price property to item types like biome ingredient recipe equipment but liam said we don't need that so

phf5kw and others added 29 commits March 2, 2025 22:02
Introduced a new `orderInstructionsContainer` to display instructions based on station ingredients. A helper method `GenerateOrderInstructions` was implemented to populate the instructions dynamically, improving clarity and guidance during gameplay.
Updated the `Order` and `Customer` classes to utilize `CorrectStockSequence` for determining recipe stock and properties. Removed redundant ingredient customization logic, making the implementation cleaner and ensuring consistency with the new recipe data structure.
Updated `OnRemoveCustomer` to handle customers by their index rather than directly referencing customer objects. Adjusted related methods, event channel, and data structures accordingly to improve reliability.
Introduced new events and methods for handling order deselection and deletion in the CookingUIEventChannel. Adjusted related UI elements and logic in StationView, CustomerController, and OrderManager to integrate these changes. Fixed bugs and improved customer spawning flow by adding delayed spawning logic.
arthuroker pushed a commit that referenced this pull request Apr 21, 2025
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.

6 participants