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

Author email and link should be optional #6

Closed
ghost opened this issue Jul 6, 2013 · 4 comments
Closed

Author email and link should be optional #6

ghost opened this issue Jul 6, 2013 · 4 comments

Comments

@ghost
Copy link

ghost commented Jul 6, 2013

I do not wish to provide email address in the feeds but the code breaks if I just omit it.

@jpmonette
Copy link
Owner

@brickcap Fixed in 3267097

Not yet updated on npm thought, so you can commit the repo or just update your ./lib/feed.js file. Thanks!

@ghost
Copy link
Author

ghost commented Jul 30, 2013

@jpmonette Can you please update the npm as well. In my application I am using feed as a dependency in package.json. So the users of my applications have to install these dependencies locally. It is not possible to manually update the libraries for every user.

@jpmonette
Copy link
Owner

@brickcap Should be good now. (btw: your project looks really interesting, I'll watch it and maybe use it :) )

@ghost
Copy link
Author

ghost commented Jul 31, 2013

Please do :) feed has been really helpful for me. I would have spent days otherwise trying to implement various syndication facilities. It is a wonderful library.

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

1 participant