-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
migrate examples to openApi part 25; affects [ctan dub hexpm jira pulsar scoop] #9749
Conversation
|
@@ -96,52 +108,58 @@ class HexPmVersion extends BaseHexPmService { | |||
} | |||
} | |||
|
|||
function DownloadsForInterval(downloadInterval) { |
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.
There is a bit of refactoring going on here.
Previously, this service used the same pattern we see in #9732 (comment)
Previously we defined a function that returns a class, and then called that function with 'day'
, 'week'
and 'all'
to generate 3 classes. One for each interval.
I don't think this really buys us much here. I converted this to a single class with an :interval(dd|dw|dt)
param. This allows us to have a single example page and show interval as a drop-down menu, which is much more in line with how we do all our other downloads badges.
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.
Makes sense!
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.
Another one that looks great! 👍🏻
Refs #9285
I wonder what number the last of these PRs will be?