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 Stock photo caption and include it when uploading a photo #15674

Closed
wants to merge 3 commits into from

Conversation

fluiddot
Copy link
Contributor

@fluiddot fluiddot commented Jan 20, 2021

Issue: wordpress-mobile/gutenberg-mobile#1609

Related PRs

Description

Add the caption field to the Stock photo upload request.

Test

The changes can be tested in two different flows:

Image block

Steps

  1. Go to a site.
  2. Open a post/page.
  3. Tap on + button.
  4. Tap on Image block.
  5. Tap on ADD IMAGE.
  6. Tap on Free Photo Library
  7. Search any term.
  8. Tap one photo.
  9. Wait for the item to be uploaded.
  10. Check that below the photo the caption field has a value.

Note: It's possible that in some cases the item doesn't have a caption value with the credit/attribution of the photo. In this case try another one.

Media screen (currently not working 🔴 due this issue)

Steps

  1. Go to a site.
  2. Tap on Media button.
  3. Tap on + button.
  4. Tap on Choose from Free Photo Library
  5. Search any term.
  6. Tap one or multiple photos.
  7. Tap on ✔️ button.
  8. Wait for the items to be uploaded.
  9. [Back in the Media screen] Tap on any of the recently uploaded photos.
  10. Check that Caption field has value.

Note: It's possible that in some cases the item doesn't have a caption value with the credit/attribution of the photo. In this case try another one.

Screenshots

Image block
image-block-caption-ios.mov
Media screen (currently not working 🔴 due this issue)
media-screen-caption-ios.mp4

PR submission checklist:

  • I have considered adding unit tests where possible.
  • I have considered adding accessibility improvements for my changes.
  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented Jan 20, 2021

You can test the changes on this Pull Request by downloading it from AppCenter here with build number: 41023. IPA is available here. If you need access to this, you can ask a maintainer to add you.

@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented Jan 20, 2021

You can trigger optional UI/connected tests for these changes by visiting CircleCI here.

@fluiddot fluiddot self-assigned this Jan 20, 2021
@fluiddot
Copy link
Contributor Author

I found out that the caption value is included in the upload request, however the site is not creating the media item with it so most likely the way we include the parameter for the caption is deprecated. I created an issue in WordPressKit-iOS repository describing the problem.

@fluiddot fluiddot force-pushed the gutenberg/add/1609-stock-photo-caption branch from bdbf157 to 8c73578 Compare January 26, 2021 17:11
@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented Jan 26, 2021

Fails
🚫 Podfile: reference to a commit hash
Warnings
⚠️ This PR is tagged with 'DO NOT MERGE'.

Generated by 🚫 dangerJS

@twstokes
Copy link
Contributor

👋 I'm going to close this PR due to lack of activity, but please reopen if this was an error. Thank you.

@twstokes twstokes closed this Aug 25, 2023
@jkmassel jkmassel deleted the gutenberg/add/1609-stock-photo-caption branch July 26, 2024 18:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants