Skip to content

Conversation

@S1v4
Copy link
Contributor

@S1v4 S1v4 commented Aug 2, 2017

No description provided.

@S1v4 S1v4 requested a review from claudiofullscreen August 2, 2017 19:28
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling a003e29 on insights into d3ccea3 on master.

lib/fb/metric.rb Outdated
@values = options[:values].map do |value_data|
OpenStruct.new(
value: value_data.fetch('value', 0),
end_time: Time.parse(value_data.fetch 'end_time')

Choose a reason for hiding this comment

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

@S1v4 replace with value_data['end_time'])

@S1v4 S1v4 force-pushed the insights branch 2 times, most recently from 6385fd3 to 2575d1b Compare August 2, 2017 21:38
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 2575d1b9d7f39938255552c5975add2b561bb1ed on insights into d3ccea3 on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 2575d1b9d7f39938255552c5975add2b561bb1ed on insights into d3ccea3 on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 230e851 on insights into d3ccea3 on master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants