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.

Sign up for GitHub

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

Jump to bottom

Gallery cover url #5182

Merged
merged 7 commits into from
Sep 3, 2024
Merged

Gallery cover url #5182

merged 7 commits into from
Sep 3, 2024

Conversation

WithoutPants
Copy link
Collaborator

  • Adds a new gallery route /gallery/:id/cover which returns the cover image data, as an alternative to getting cover via graphql.
  • Adds cover member to GalleryPathsType so that this URL is included in the returned paths.
  • Adds a default gallery image for when the gallery has no images.
  • Fixes issue with the gallery preview scrubber when gallery has no images
  • Fixes issue with lightbox hanging when clicking on a gallery in the gallery wall when the gallery has no images
  • Documents a known issue with the DLNA server where shutting the server down leaves remnant goroutines.

@WithoutPants WithoutPants added bug Something isn't working improvement Something needed tweaking. labels Aug 29, 2024
@WithoutPants WithoutPants added this to the Version 0.27.0 milestone Aug 29, 2024
@WithoutPants WithoutPants merged commit a3c34a5 into stashapp:develop Sep 3, 2024
2 checks passed
damontecres added a commit to damontecres/StashAppAndroidTV that referenced this pull request Sep 5, 2024
@damontecres
Use gallery.path.cover for cover image
b2efb62 
Ref: stashapp/stash#5182
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
bug Something isn't working improvement Something needed tweaking.
Projects
None yet
Milestone
Version 0.27.0
Development

Successfully merging this pull request may close these issues.
1 participant
@WithoutPants