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

Fixed RSS error for :all episodes #128

Merged
merged 1 commit into from
Mar 31, 2015

Conversation

kirushik
Copy link
Contributor

Closes #126

But I really don't like duplication between ApplicationController#set_episode and ProjectsController#load_episode

Closes SUSE#126

But I really don't like duplication between
`ApplicationController#set_episode` and
`ProjectsController#load_episode`
@coveralls
Copy link

Coverage Status

Coverage increased (+0.39%) to 78.25% when pulling 3d3e9ab on kirushik:fix_for_RSS_feeds into af11a92 on SUSE:master.

@hennevogel
Copy link
Member

Me neither. The whole episode handling isn't my favourite hack...

hennevogel added a commit that referenced this pull request Mar 31, 2015
Fixed RSS error for :all episodes
@hennevogel hennevogel merged commit ab600af into SUSE:master Mar 31, 2015
@kirushik kirushik deleted the fix_for_RSS_feeds branch March 31, 2015 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

RSS without episode or with 'all' string
3 participants