Skip to content
This repository has been archived by the owner on May 11, 2020. It is now read-only.

Restyle Updated map to implement the stock market #46

Merged
merged 1 commit into from
Dec 20, 2019

Conversation

restyled-io[bot]
Copy link
Contributor

@restyled-io restyled-io bot commented Dec 18, 2019

Automated style fixes for #45, created by Restyled.

The following restylers made fixes:

  • whitespace

To incorporate these changes, merge this Pull Request into the original. We
recommend using the Squash or Rebase strategies.

NOTE: As work continues on the original Pull Request, this process will
re-run and update (force-push) this Pull Request with updated style fixes as
necessary. If the style is fixed manually at any point (i.e. this process finds
no fixes to make), this Pull Request will be closed automatically.

Sorry if this was unexpected. To disable it, see our documentation.

@always-be-closing always-be-closing bot mentioned this pull request Dec 18, 2019
4 tasks
@restyled-io restyled-io bot force-pushed the stock-market-restyled branch 2 times, most recently from 42e1a6d to 65ca73d Compare December 19, 2019 09:37
@Seniru
Copy link
Owner

Seniru commented Dec 20, 2019

Approved

@Seniru Seniru merged commit 8c4e6bc into stock-market Dec 20, 2019
@Seniru Seniru deleted the stock-market-restyled branch December 20, 2019 06:36
Seniru added a commit that referenced this pull request Dec 20, 2019
* Updated map to implement the stock market

* ui: Set all text areas to a fixed position

* Added linechart to the map

* feat: Added a time system

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

* Basic support for multi owners for a company

* fix: Fixed problems with receiving dividends and investing

* style: Restyled by whitespace (#46)

* Improved how salary and energy prepared for a job

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

* Restyled by whitespace (#52)

* Supported a player to own and create 1 company

* Supported functioning of dashboard, bug fixes and improvements

- 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

Co-authored-by: restyled-io[bot] <32688539+restyled-io[bot]@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants