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

Cleanup the Status View #5317

Merged
merged 9 commits into from
Mar 29, 2017
Merged

Cleanup the Status View #5317

merged 9 commits into from
Mar 29, 2017

Conversation

ngotchac
Copy link
Contributor

@ngotchac ngotchac commented Mar 28, 2017

This PR cleans up the Status page, which would be needed for #4992

  • Put the dev logs in a MobX store, and fix the clear functionality which wasn't really clearing the logs
  • Remove what can be remove from Redux about the status (ie. data only on the status view) and put it back in a MobX store
  • Use a proper Address Selector for the coinbase field
  • Have a way to reset the extra data (press ESC to reset to default)
  • Delete all the un-needed files
  • Move the un-included RPC Calls related files to a separate folder (until implemented) (views/RpcCalls)
  • Add the CopyToClipboard capability to non-readOnly address inputs (eg. Status > Coinbase)
  • Show the list of Peers in the Status Page

@ngotchac ngotchac added A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. M7-ui labels Mar 28, 2017
@ngotchac ngotchac added A0-pleasereview 🤓 Pull request needs code review. and removed A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. labels Mar 28, 2017
@jacogr jacogr added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Mar 29, 2017
@gavofyork gavofyork merged commit 5fa0881 into master Mar 29, 2017
@gavofyork gavofyork deleted the ng-status-update branch March 29, 2017 12:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants