github-mirror/discourse
2
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2024-11-30 08:03:40 +08:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

Merge pull request #5510 from discourse/add-posts-count-to-web-hook-post-serializer

Browse Source 
Feature: add topic posts count to webhook post serializer
This commit is contained in:
Guo Xiang Tan 2018-01-23 07:46:59 +08:00 committed by GitHub
commit 80c41a0e59
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 9 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
app/serializers/web_hook_post_serializer.rb
View File

@ -1,4 +1,7 @@
class WebHookPostSerializer < PostSerializer class WebHookPostSerializer < PostSerializer
attributes :topic_posts_count
def include_topic_title? def include_topic_title?
true true
end end
@ -14,4 +17,9 @@ class WebHookPostSerializer < PostSerializer
false false
end end
end end
def topic_posts_count
object.topic ? object.topic.posts_count : 0
end
end end

2
spec/serializers/web_hook_post_serializer_spec.rb
View File

@ -7,7 +7,7 @@ RSpec.describe WebHookPostSerializer do
it 'should only include the required keys' do it 'should only include the required keys' do
count = serializer.as_json.keys.count count = serializer.as_json.keys.count
difference = count - 40 difference = count - 41
expect(difference).to eq(0), lambda { expect(difference).to eq(0), lambda {
message = "" message = ""