Fix errors in the atom feed #228

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
Contributor

cgriego commented Oct 11, 2010

Originally this issue was opened
http://github.com/rubygems/gemcutter/issues/181

And this commit was merged but it did not fix the problem as outlined in the comments
http://github.com/rubygems/gemcutter/commit/985ad307024417653bc179103829ab6c89e431d6

This pull request fixes the following issues:

  • Add platform in title if needed
  • Fix undefined entry element: platform validation error
  • Fix link url to point to the specific rubygem version (currently points to /gems/rspec?format=2.0.0)
  • Fix multiple entries with the same id validation error (currently points to /gems/rspec?version=2.0.0)
  • Fix missing author element: name validation error (also fixes the authors text not getting output)
  • Fix summary should not be blank validation warning
  • Fix double escaped version summary
Fix errors in the Atom feed
* Add platform in title if needed
* Fix undefined entry element: platform validation error
* Fix link url to point to the specific rubygem version
* Fix multiple entries with the same id validation error
* Fix missing author element: name validation error
* Fix summary should not be blank validation warning
* Fix double escaped version summary
Owner

qrush commented Nov 5, 2010

Merged, thanks!!

This issue was closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment