Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Refactoring app/services/exporter/articles.rb #5094

Merged
merged 1 commit into from Dec 13, 2019
Merged

Refactoring app/services/exporter/articles.rb #5094

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
34 changes: 24 additions & 10 deletions app/services/exporter/articles.rb
View file
Open in desktop
Expand Up @@ -18,37 +18,51 @@ def export(slug: nil)
private

def allowed_attributes
time_attributes | url_attributes | general_attributes
end

def time_attributes
%i[
created_at
crossposted_at
edited_at
last_comment_at
published_at
]
end

def url_attributes
%i[
canonical_url
feed_source_url
video_closed_caption_track_url
video_source_url
video_thumbnail_url
]
end

def general_attributes
%i[
body_markdown
cached_tag_list
cached_user_name
cached_user_username
canonical_url
comments_count
created_at
crossposted_at
description
edited_at
feed_source_url
language
last_comment_at
main_image
main_image_background_hex_color
path
positive_reactions_count
processed_html
published
published_at
published_from_feed
show_comments
slug
social_image
title
video
video_closed_caption_track_url
video_code
video_source_url
video_thumbnail_url
]
end

Expand Down