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.

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

Possibly resolves #42 #75

Open
wants to merge 2 commits into
base: devel
Choose a base branch
from
Open

Possibly resolves #42 #75

wants to merge 2 commits into from

Commits on Nov 30, 2016

  1. shows.json can now contain a "limit" member that specifies a characte…

    …r limit for the specific show's title.
    
    Changed hard limit in DataMapper class in suggestion.rb from 40 to the value of live show's limit. Not sure of workflow. May be a problem there is no live show in data.json?
    Also bumped setting LIVE_URL to immediately after call to Dotenv.load, otherwise it gets set too late to
    initialize the Suggestion class.
    
    shows.json example:
    
    {
      "shows": [
        {
          "rss":"http://feeds.feedburner.com/BsdNowHd",
          "url": "bn",
          "title": "BSD Now"
        },
        {
          "rss":"http://feeds.feedburner.com/coderradiovideo",
          "url": "cr",
          "title": "Coder Radio",
          "limit": 10
        }
      ]
    }
    ObserverHerb committed Nov 30, 2016
    Configuration menu
    Copy the full SHA
    b9cf624 View commit details
    Browse the repository at this point in the history

Commits on Dec 3, 2016

  1. Additional testing revealed that if data.json exists but contains inv…

    …alid slug, suggestions.rb crashes. Added a condition for when the live show cannot be found so limit defaults to 40.
    ObserverHerb committed Dec 3, 2016
    Configuration menu
    Copy the full SHA
    dabe9bc View commit details
    Browse the repository at this point in the history