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
Added 'metacpan_url' method for fetching MetaCPAN links of module, re… #74
Conversation
A dist is |
Yeah, I think adding one for distribution would be nice. |
the above suggested link for distribution will give the release (latest) page which can be a bit confusing given the data we hold in the Distribution object in the client (which reflects the distribution type in our ES data). |
It looks like the module link is wrong in the code. |
4430d2e
to
c921b6f
Compare
fixed (i hope) and added distirbution |
c921b6f
to
343adb1
Compare
examples/metacpan_url.pl
Outdated
use strict; | ||
use warnings; | ||
use MetaCPAN::Client; | ||
use DDP; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think DDP
is actually being used?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
right, that's just part of my template :) will clean it out
Supports: Module, Release, Distribution, File & Author.
343adb1
to
d3297e6
Compare
#69
added 'metacpan_url' method to Module, Release and Author types.
Is anything missing? (I don't think Distribution has a representation page on the frontend)