Issue #30: Add getMiningStatus to src/actions/index.js

bencassidy commented 6 years ago

Description

Added call to getMiningStatus in src/actions/index.js

Related Issue

Fixes #30

Motivation and Context

Step towards displaying the current status of mining at the top section of the application

How Has This Been Tested?

No functional changes as far as I can tell, so I didn't test anything on xplore. I ran the jest tests and they passed.

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

griffobeid commented 6 years ago

This looks great @bencassidy but there is one issue. It doesn't look like your fork is up to date with the project and that causes an issue when merging. Take a look at this gist, it makes staying up to date with the project easy.

Otherwise, everything looks great! :+1:

bencassidy commented 6 years ago

@griffobeid conflicts resolved 👍

griffobeid commented 6 years ago

Thanks, @bencassidy and congrats on your first pull request! :tada: 👍 I hope to see more contributions for you 😄

DAVFoundation / xplore