Skip to content
This repository was archived by the owner on Sep 15, 2025. It is now read-only.

Conversation

@jklausa
Copy link
Contributor

@jklausa jklausa commented Feb 22, 2019

Description

This should be the last one of those!

Regretfully, this and #101 have some of the most weird and complicated logic for parsing the data — I think one quick glance at the attached sample data will speak for itself — and hopefully explains the sins commited in this PR :P

Testing Details

  • Verify that the lib builds
  • Verify the tests pass
  • Please check here if your pull request includes additional test coverage.

@jklausa jklausa self-assigned this Feb 22, 2019
@jklausa jklausa requested a review from ScoutHarris February 22, 2019 00:31
Copy link
Contributor

@ScoutHarris ScoutHarris left a comment

Choose a reason for hiding this comment

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

:shipit:

@jklausa jklausa merged commit 520e927 into develop Feb 22, 2019
@jklausa jklausa deleted the feature/stats-fetch-post-details branch February 22, 2019 21:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants