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

extend UnsplashBridge (II) #1813

Merged
merged 11 commits into from
Apr 5, 2022
Merged

extend UnsplashBridge (II) #1813

merged 11 commits into from
Apr 5, 2022

Conversation

langfingaz
Copy link
Contributor

@langfingaz langfingaz commented Oct 21, 2020

Rebased pull request #1744.

Summary of changes:

  • filter photo stream for username
  • change image preview quality
  • fix caching of rss entries by setting '$item['timestamp']'
  • add link to user, location (if present), link to photo on unsplash to $item['content']

Thanks @nel50n for his UnsplashBridge! This Commit is mainly based on his work.

Signed-off-by: langfingaz <[email protected]>
+ filter photo stream for username
+ change image preview quality
+ add link to user, location (if present), link to photo on unsplash to $item['content']

Signed-off-by: langfingaz <[email protected]>
Functionality is now merged into UnsplashBridge
@em92
Copy link
Contributor

em92 commented Oct 30, 2020

Hi again, @langfingaz! Sorry for long delay. Could you lint your code please for travis to show green tick instead of red cross?
https://travis-ci.com/github/RSS-Bridge/rss-bridge/jobs/403152648

@github-actions
Copy link

github-actions bot commented Mar 25, 2022

Pull request artifacts

file last change
Unsplash-current-context1 2022-04-05, 12:59:15
Unsplash-pr-context1 2022-04-05, 12:59:15

github-actions bot added a commit that referenced this pull request Mar 25, 2022
github-actions bot added a commit that referenced this pull request Mar 25, 2022
Copy link
Contributor

@Bockiii Bockiii left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no 'required' for lists

bridges/UnsplashBridge.php Outdated Show resolved Hide resolved
github-actions bot added a commit that referenced this pull request Mar 26, 2022
github-actions bot added a commit that referenced this pull request Mar 26, 2022
@Bockiii
Copy link
Contributor

Bockiii commented Mar 26, 2022

Hi,

I fixed the merge issues, please check if the bridge still does what it should do @langfingaz .

Additionally, please use either exampleValue OR defaultValue for a given parameter, not both. Also, please use a value that would generate an actual feed (from the current username example "unsplash" will produce something, while "someuser" will probably not).

bridges/UnsplashBridge.php Outdated Show resolved Hide resolved
bridges/UnsplashBridge.php Outdated Show resolved Hide resolved
this is just to kick off another round of checks.
bridges/UnsplashBridge.php Outdated Show resolved Hide resolved
@Bockiii Bockiii merged commit 6ffe531 into RSS-Bridge:master Apr 5, 2022
Kwbmm pushed a commit to Kwbmm/rss-bridge that referenced this pull request Jun 17, 2022
@langfingaz langfingaz deleted the unsplash branch September 30, 2022 18:16
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.

3 participants