Skip to content
This repository has been archived by the owner on Sep 20, 2023. It is now read-only.

Added new miner Command #215

Merged
merged 20 commits into from
Jan 19, 2021
Merged

Added new miner Command #215

merged 20 commits into from
Jan 19, 2021

Conversation

Tert0
Copy link
Member

@Tert0 Tert0 commented Jan 19, 2021

Description
Added a miner command in the terminal.
You can create miners and change the corresponding wallet and power.
You can also change miners of other (hacked) users.

Issue
None

Testing Instructions
Type "miner" in the Terminal app and you will get the syntax hint.

@Tert0
Copy link
Member Author

Tert0 commented Jan 19, 2021

How can I add a new Commit to this Pull Request?

@Tert0 Tert0 closed this Jan 19, 2021
@Tert0 Tert0 reopened this Jan 19, 2021
@JannikEmmerich
Copy link
Member

How can I add a new Commit to this Pull Request?

Just push the new commits to Tert0:master.

@Tert0
Copy link
Member Author

Tert0 commented Jan 19, 2021

Now its works but it Bugged.
After the reopen it worked

Copy link
Member

@JannikEmmerich JannikEmmerich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove commit ceb9d08 since it has nothing to do with your pull request.

Remove all the changes that are not required for the feature.

This pull request should be squashed to Added new miner command (#215)

src/app/desktop/windows/terminal/terminal-states.ts Outdated Show resolved Hide resolved
src/app/desktop/windows/terminal/terminal-states.ts Outdated Show resolved Hide resolved
src/app/desktop/windows/terminal/terminal-states.ts Outdated Show resolved Hide resolved
src/app/desktop/windows/terminal/terminal-states.ts Outdated Show resolved Hide resolved
src/app/desktop/windows/terminal/terminal-states.ts Outdated Show resolved Hide resolved
@Tert0
Copy link
Member Author

Tert0 commented Jan 19, 2021

@JannikEmmerich Tried to fx all the Problems.I hope that's good

@JannikEmmerich
Copy link
Member

@JannikEmmerich Tried to fx all the Problems.I hope that's good

Remove commit ceb9d08 since it has nothing to do with your pull request.

@Tert0
Copy link
Member Author

Tert0 commented Jan 19, 2021

How to remove a Commit from an Pull Request? @JannikEmmerich

@JannikEmmerich
Copy link
Member

How to remove a Commit from an Pull Request? @JannikEmmerich

You can either revert it or remove it from the commit history.

@Tert0
Copy link
Member Author

Tert0 commented Jan 19, 2021

How to remove a Commit from an Pull Request? @JannikEmmerich

You can either revert it or remove it from the commit history.

Where? ;)

JannikEmmerich
JannikEmmerich previously approved these changes Jan 19, 2021
@JannikEmmerich
Copy link
Member

This pull request has to be squashed to Added new miner command (#215).

Tert0 and others added 3 commits January 19, 2021 19:18
Copy link
Member

@MaxiHuHe04 MaxiHuHe04 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good job! Just a few minor adjustments.

src/app/desktop/windows/terminal/terminal-states.ts Outdated Show resolved Hide resolved
src/app/desktop/windows/terminal/terminal-states.ts Outdated Show resolved Hide resolved
src/app/desktop/windows/terminal/terminal-states.ts Outdated Show resolved Hide resolved
@MaxiHuHe04 MaxiHuHe04 merged commit 2dea7ab into cryptic-game:master Jan 19, 2021
@Tert0 Tert0 mentioned this pull request Jan 21, 2021
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.

4 participants