update to embed code

[Heather Piwowar]

Hi all, 

We pushed out an update to the embed code this morning.  It no longer polls, waiting for total-impact to finish collecting metrics: the total-impact api now tells the embed code when it has finished collecting metrics for an item (response code 210 when updating, response code 200 when updating complete).

It should work as before, but be a bit faster.  Let us know if you run into any problems.

Heather

[Jason Priem]

sweet, hadn't read this! this is pretty awesome.

you should brace for more emails...have been in mostly-ignore-email mode until I fixed that bug...so now in mostly-do-email mode :)

j

--
Jason Priem
UNC Royster Scholar
School of Information and Library Science
University of North Carolina at Chapel Hill

[Mark Wood]

Thanks!

I tried insinuating this into a DSpace instance, to put the metrics near the bottom of item detail pages.  Either I'm doing something wrong (not unlikely) or the CSS needs some more specificity:  all the links on the page turned large and orange.  I think that all the rules need a "#ti-data" ancestor.

[Mark Wood]

|  I think that all the rules need a "#ti-data" ancestor.

Like this.  Not really tested yet:  it no longer munges the rest of the page, but now I need to put some content into the repo. I'm testing with so I can see the metrics div.

[Jason Priem]

Awesome, Mark!  I'm travel all tomorrow, but will try to test out your code. Looks good from here. Let me know if you test it and it works with a pull request, and I'll merge it in right away. 

Heather, sorry did not get more time to work on this today. 

J

Sent from my telephone