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

Update #17

Merged
merged 2 commits into from
Aug 7, 2016
Merged

Update #17

merged 2 commits into from
Aug 7, 2016

Conversation

kimisan
Copy link
Member

@kimisan kimisan commented Aug 7, 2016

  1. add a new function, call_hashrate_chart() to append D3.js chart
  2. modify the 780G --> 780GH/s

<script>
call_hashrate_chart();
</script>

</html>
Copy link
Collaborator

Choose a reason for hiding this comment

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

this file isn't actually being used... I moved it to public/views/stats/index.html
So all I need to do is call_hashrate_chart() from the StatsController, right?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes

Copy link
Collaborator

Choose a reason for hiding this comment

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

Where is hashrate.js included? Right now call_hashrate_chart() is undefined

@elaineo elaineo merged commit d162beb into ethereumclassic:master Aug 7, 2016
@elaineo
Copy link
Collaborator

elaineo commented Aug 7, 2016

oops, ignore my earlier comments, i forgot to move the files to the proper location. sorry!

@kimisan
Copy link
Member Author

kimisan commented Aug 8, 2016

haha,
By the way, for the http://etherscan.io/stats/miner?range=7, if the API is ready, please let me know
And If you have some new visualization idea or assignment , please also let me know (`・ω・´)

@elaineo
Copy link
Collaborator

elaineo commented Aug 8, 2016

i'm sure the ppl on reddit have tons of ideas if you ask them ;)

On Sun, Aug 7, 2016 at 10:41 PM, Kimi Sian-yu Chen <notifications@github.com

wrote:

haha,
By the way, for the http://etherscan.io/stats/miner?range=7, if the API
is ready, please let me know
And If you have some new visualization idea or assignment , please also
let me know (`・ω・´)


You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
#17 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACPZr30gfU5BRgXu9K7H1eMTV5PVHbyuks5qdsGigaJpZM4JefGt
.

@kimisan
Copy link
Member Author

kimisan commented Aug 8, 2016

haha ,Good idea!

phyro pushed a commit that referenced this pull request Apr 15, 2019
Remove author from html tags
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants