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

Paycheck Code[Experimental Testmerge] #2294

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

InsightfulParasite
Copy link
Collaborator

@InsightfulParasite InsightfulParasite commented Jul 10, 2024

About The Pull Request

Using the economy system i made a proc for paying employees on a 20min interval. Honestly id just start from scratch rather than using this EXTREMELY bloated economy system.

Why It's Good For The Game

Requested by my fellow maintainers. Kind of unsure about this code since when i asked "Why" i was told that money wasnt used for much of anything currently. UPDATE. Was told this was a foundational code for making money less connected to gatcha rewards.

Changelog

🆑
add: paychecks
/:cl:

@InsightfulParasite InsightfulParasite added Feature Something new being added. Isolated Code The files added or edited by this PR have a low chance of conflicts but still need to be reviewed. Do Not Merge - Maintainer Discussion Do not merge until Maintainers talk about it. labels Jul 10, 2024
@Coxswain-Navigator
Copy link
Contributor

The "Why" is pretty much so that we can put an end to rewarding players with ahn for some of the random jobs they can do. Like being unlucky with a gacha. There are actually a few things I've proposed making players use their personal funds for, like life insurance just as an example. Think of this as a foundation.

@InsightfulParasite
Copy link
Collaborator Author

The "Why" is pretty much so that we can put an end to rewarding players with ahn for some of the random jobs they can do. Like being unlucky with a gacha. There are actually a few things I've proposed making players use their personal funds for, like life insurance just as an example. Think of this as a foundation.

Makes sense. Ill polish the code later when im more awake.

@InsightfulParasite InsightfulParasite force-pushed the payrollsystem branch 3 times, most recently from e802917 to 8fda0fd Compare July 11, 2024 02:32
@InsightfulParasite InsightfulParasite marked this pull request as ready for review July 12, 2024 07:47
@InsightfulParasite InsightfulParasite force-pushed the payrollsystem branch 2 times, most recently from fbff3e2 to 0303402 Compare July 14, 2024 09:00
@InsightfulParasite InsightfulParasite changed the title Paycheck Code Paycheck Code[Testmerge Done] Jul 26, 2024
@InsightfulParasite InsightfulParasite changed the title Paycheck Code[Testmerge Done] Paycheck Code[Experimental Testmerge] Jul 26, 2024
@InsightfulParasite InsightfulParasite force-pushed the payrollsystem branch 2 times, most recently from e1ce398 to 0d49c92 Compare December 7, 2024 23:23
Using the economy system i made a proc for paying employees on a 20min interval. Honestly id just start from scratch rather than using this EXTREMELY bloated economy system.

add: paychecks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Do Not Merge - Maintainer Discussion Do not merge until Maintainers talk about it. Feature Something new being added. Isolated Code The files added or edited by this PR have a low chance of conflicts but still need to be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants