Skip to content
This repository has been archived by the owner on Oct 1, 2019. It is now read-only.

Recent tags (API) #965

Merged
merged 15 commits into from
Dec 15, 2017
Merged

Recent tags (API) #965

merged 15 commits into from
Dec 15, 2017

Conversation

voidxnull
Copy link
Member

@voidxnull voidxnull commented Nov 19, 2017

#951

  • Add getRecentlyUsedTags controller (/api/v1/recent-tags). Cached with redis.
  • Implement redis caching.
  • Add redux store.
  • Add controller tests.
  • Extend allPosts:
    • total post count (totalCount query param)
    • tagType filter
    • dateRange filter
  • Add counters to getRecentlyUsedTags.

@voidxnull voidxnull force-pushed the feature/recent-tags branch 2 times, most recently from 383f738 to 13bc944 Compare November 26, 2017 22:01
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 13bc944 on feature/recent-tags into ** on stable**.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling a49e5fb on feature/recent-tags into ** on stable**.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 6cf2296 on feature/recent-tags into ** on stable**.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 6cf2296 on feature/recent-tags into ** on stable**.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.07%) to 58.552% when pulling 353bbd5 on feature/recent-tags into 446da30 on stable.

Extracted most of the code to {TagType}.getRecentlyUsed so it can
be reused.
@voidxnull voidxnull changed the title [WIP] Recent tags Recent tags Dec 6, 2017
@voidxnull voidxnull changed the title Recent tags Recent tags (API) Dec 6, 2017
@coveralls
Copy link

Coverage Status

Coverage increased (+0.05%) to 58.532% when pulling a227215 on feature/recent-tags into 446da30 on stable.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.05%) to 58.532% when pulling a227215 on feature/recent-tags into 446da30 on stable.

@voidxnull voidxnull changed the title Recent tags (API) [WIP] Recent tags (API) Dec 8, 2017
@voidxnull voidxnull force-pushed the feature/recent-tags branch 2 times, most recently from 91f5b4b to b15d32d Compare December 10, 2017 13:21
@coveralls
Copy link

Coverage Status

Coverage increased (+0.07%) to 58.544% when pulling b15d32d on feature/recent-tags into 446da30 on stable.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.07%) to 58.544% when pulling 486d097 on feature/recent-tags into 446da30 on stable.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.07%) to 58.544% when pulling 486d097 on feature/recent-tags into 446da30 on stable.

@voidxnull voidxnull force-pushed the feature/recent-tags branch 3 times, most recently from 13c6de4 to a9009b8 Compare December 11, 2017 05:06
@voidxnull voidxnull changed the title [WIP] Recent tags (API) Recent tags (API) Dec 11, 2017
Changed the signature of the applySomeQuery functions to be more
intuitive and readable.
Removed unnecessary async annotations.
Fixed possibility of SQL injection.
Added table option.
Allows user to retrieve a total number of queried posts ignoring
LIMIT and OFFSET. Opt-in for performance reasons.
Some tests check the order of schools. Comparing strings in
js and pg yields different results.
@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 58.574% when pulling 5c307de on feature/recent-tags into 446da30 on stable.

Copy link
Member

@artkravchenko artkravchenko left a comment

Choose a reason for hiding this comment

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

Thank you!

@artkravchenko artkravchenko merged commit 8422bb7 into stable Dec 15, 2017
@artkravchenko artkravchenko deleted the feature/recent-tags branch December 15, 2017 08:31
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.

None yet

3 participants