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

Error in Documentation #7

Closed
f10i opened this issue Jul 11, 2009 · 1 comment
Closed

Error in Documentation #7

f10i opened this issue Jul 11, 2009 · 1 comment

Comments

@f10i
Copy link

f10i commented Jul 11, 2009

I was trying to use feedzirra with the on_success callback, but kept getting the following message:
warning: multiple values for a block parameter (2 for 1), and the code would jump to the on_failure block.

Checking the source code for feed.rb, shows that the on_success callback accepts two parameters, url and feed, whereas the documentation only states feed.

An update to the documentation might be in order. Thank you for the great work

@stve
Copy link

stve commented Oct 26, 2009

It should be

:on_success => lambda {|url, feed| puts feed.title }

instead of:

:on_success => lambda {|feed| puts feed.title }

PascalTurbo pushed a commit to PascalTurbo/feedjira that referenced this issue Dec 19, 2016
# This is the 1st commit message:
Added core logging functionality (feedjira#312)
Added core configuration functionality (feedjira#340)

# This is the commit message #2:

Add convenience Methods for logging Messages (feedjira#312)

# This is the commit message feedjira#3:

Replace warn with logger (feedjira#312)

# This is the commit message feedjira#4:

Add logger messages for shadowed exceptions (feedjira#312)

# This is the commit message feedjira#5:

Enhaced Documentation for LogLevel (feedjira#312)

# This is the commit message feedjira#6:

Fixed some Logger related issues (feedjira#312)

# This is the commit message feedjira#7:

Added local DEFAULTS configuration (feedjira#312,feedjira#340)
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

No branches or pull requests

3 participants