EFForg / EFForg/sec

Use predictable column names for content

Open
#391 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
40
Forks
9
PR merge metrics
No merged PRs in 30d

Description

`ContentHelper#preview` is getting a little complex in how it finds a piece of content's html:

https://github.com/EFForg/sec/blob/242e1ec95608130f08d13d493da8b6f6434b6e0f/app/helpers/content_helper.rb#L7-L13

Currently articles, blog_posts, lessons, and managed_content use "body" as their main text field, while materials, topics, and uploads use "description". In addition, articles has a "summary" field that the rest lack. It would be nice if we migrated these tables so they all use the same name for this kind of column.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.