This repository has been archived by the owner on May 11, 2020. It is now read-only.
Add support for stock market #45
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Seniru
changed the title
Seniru
added
affects: structure
feature
Map
|
At this point (with the stock market dashboard), the map looks like this
|
With this feature, - a new day occur with every second, and months and years accordinly, - Added a clock in the map to display the time closes #47
As some players can create high salary jobs with a low energy, it can become unfair to other players. To overcome this we introduced a new way to prepare the salary. Accordingly, - High salary increases the energy used, and vice-versa - Jobs with high level requirements can reduce the energy level used - Jobs with high educational level requirements can reduce the the energy level used This can also be considered as some additional solutions for issue #29
- Fixed error caused due to passing a nil value inside the timer - Improved the Player:addOwnerCompanies method so that new companies are added only if the company doesn't exist in player owned company list. - Supported functioning of stock market dashboard. It updates every month according to the game - Minor improvements
|
We are happy to announce that the dashboard is functioning now (after 9767ac9). Here are some screenshots
|
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR will add a new feature to the game - that is the stock market. With this feature, we are going to introduce support to buy shares or invest in other people's companies, receive dividends (not only for the owner but also the shareholders), and many things related with real business world. To implement these changes we identified the following as some goals to fulfill.
This closes #50 and closes #47