PIXEL selling and buyEnergy #706
Open
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
Contributor
mschultheiss83
commented
Feb 26, 2023
mschultheiss83
commented
- cleanup pixel code, sell PIXELs
- buyEnergy if we have terminals and storage, mineral creeps should move energy to storage for builders and upgrader (see PR fixed: mineral handling #703)
- fixed quests.js for incoming transactions
- buyEnergy if we have terminals and storage, mineral creeps should move energy to storage for builders and upgrader sell PR TooAngel#703 - fixed quests.js for incoming transactions
TooAngel
requested changes
Feb 26, 2023
| @@ -0,0 +1,40 @@ | |||
| 'use strict'; | |||
There was a problem hiding this comment.
I'm not sure about the brain prefix anymore. I would just call it pixel.js
| // 300M credits | ||
| minCredits: 300000000, | ||
| // 300K credits | ||
| minCredits: 300000, |
There was a problem hiding this comment.
I think we should have as minCredits the price of an access key, which is currently around 500M.
So we shouldn't reduce this, if you prefer you can change it in your local config.
What do you think?
| */ | ||
| function checkQuestForAcceptance(transaction) { | ||
| Memory.quests = Memory.quests || {}; | ||
| transaction.description = transaction.description || JSON.stringify({ |
There was a problem hiding this comment.
I don't understand this change.
This sets the description of the transaction to some 'random' value.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.