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

Add constructor to SubredditPaginator with parameter for subreddit #1

Merged
merged 1 commit into from
Sep 10, 2014
Merged

Conversation

mtt88
Copy link
Contributor

@mtt88 mtt88 commented Sep 6, 2014

New constructor in SubredditPaginator to simplify paging over a subreddit.
Change the way we pull username/pass in unit tests to split correctly on Windows too.

@mattbdean
Copy link
Owner

Thanks for your commit. The Travis build failed, but I don't think that's because of your changes. I'll run the tests again once I can get my laptop working properly.

@mtt88
Copy link
Contributor Author

mtt88 commented Sep 8, 2014

It appears to be due security restrictions that Travis imposes on secure environment variables when handling pull requests from a forked repo. Not sure the best way to handle this scenario - this is the first time I've worked with Travis. I'll read into it.

Matt

ref : http://docs.travis-ci.com/user/pull-requests/

@mattbdean
Copy link
Owner

It worked on my Raspberry Pi, so I'll go ahead and merge it. I'll have to change the unit tests/gradle config so that tests that require a username/password aren't run for pull request builds.

mattbdean added a commit that referenced this pull request Sep 10, 2014
Add constructor to SubredditPaginator with parameter for subreddit
@mattbdean mattbdean merged commit 2d484aa into mattbdean:master Sep 10, 2014
fbis251 pushed a commit to fbis251/JRAW that referenced this pull request Apr 28, 2016
dukestreet pushed a commit to dukestreet/JRAW that referenced this pull request Feb 13, 2022
* Submission: is_gallery field

* Submission: gallery_data field

* GalleryData adapter

* Submission: media_metadata field

* Enable moshi kapt codegen

* Add original to MediaMetadata

* Revert "GalleryData adapter"

This reverts commit df64b2c.

* Use generated adapter for GalleryData

* Add caption field for gallery

* Support for gifs in media_metadata

* media_metadata: some fields are optional
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

Successfully merging this pull request may close these issues.

2 participants