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

Supported investments and changed the way salary is prepared and get #43

Merged
merged 6 commits into from
Dec 17, 2019

Conversation

Seniru
Copy link
Owner

@Seniru Seniru commented Dec 17, 2019

This PR addressed and closes #29 where new ways of making salaries are discussed

The changes with this PR can be summarized as follows

  • Adding support to investments
  • Changing the maximum salary paid by a company to the 1/10th of it's capital
  • Increasing the salary of a player according to his/her education level. This change was implemented to give a value to education (which had no much value previously)

TODO:

  • Improving the UI of the Investment (most probably a new button)

The salary counted as follows,

salary = basic + basic * eduLvl + 0.1

This functionality was introduced to add a value to the education as
discussed in #29
@always-be-closing
Copy link

6624e51 restyled — Restyling found differences Details

@Seniru Seniru changed the title :sonstruction: Supported investments and changed the way salary is prepared and get 🚧 Supported investments and changed the way salary is prepared and get Dec 17, 2019
@Seniru Seniru changed the title 🚧 Supported investments and changed the way salary is prepared and get 🚧 Supported investments and changed the way salary is prepared and get Dec 17, 2019
@Seniru Seniru changed the title 🚧 Supported investments and changed the way salary is prepared and get Supported investments and changed the way salary is prepared and get Dec 17, 2019
@Seniru Seniru merged commit dcec641 into master Dec 17, 2019
@Seniru Seniru deleted the Investments branch December 17, 2019 09:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Job-making is too OP
1 participant