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 /works/{id}/thumbnail route #15

Merged
merged 1 commit into from
Aug 25, 2022
Merged

Conversation

mbklein
Copy link
Contributor

@mbklein mbklein commented Aug 24, 2022

To test:

  1. Run Meadow in your dev environment
  2. Create one or more works
  3. Assign representative images
  4. Make sure both the Indexer and the Reindexer complete a pass before you shut down Meadow
  5. Start the API locally
  6. Request /works/[work-id]/thumbnail for each of the works you created in step 2
  7. Request thumbnails for works that don't exist to make sure they return 404
  8. If you want to be extra thorough, create some corrupt work records in the work index with bad or missing thumbnail URLs and try to request them

@mbklein mbklein requested review from bmquinn and kdid August 24, 2022 19:02
@mbklein mbklein force-pushed the 3113-get-work-thumbnail branch 2 times, most recently from eee5f99 to 5aec4f2 Compare August 24, 2022 19:18
@bmquinn bmquinn merged commit 0a458de into deploy/staging Aug 25, 2022
@bmquinn bmquinn deleted the 3113-get-work-thumbnail branch August 25, 2022 17:54
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