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

Use AMP_Validated_URL_Post_Type::get_url_from_post() instead of post_title property #6764

Merged
merged 3 commits into from
Dec 7, 2021

Conversation

dhaval-parekh
Copy link
Collaborator

Summary

Related #6707
Amend #6718

Checklist

  • My code is tested and passes existing tests.
  • My code follows the Engineering Guidelines (updates are often made to the guidelines, check it out periodically).

@dhaval-parekh dhaval-parekh self-assigned this Dec 6, 2021
@github-actions
Copy link
Contributor

github-actions bot commented Dec 6, 2021

Plugin builds for 969a3ed are ready 🛎️!

@westonruter westonruter added this to the v2.2 milestone Dec 6, 2021
@dhaval-parekh dhaval-parekh changed the title Use AMP_Validated_URL_Post_Type::get_url_from_post() instead of ->post_title Use AMP_Validated_URL_Post_Type::get_url_from_post() instead of post_title property Dec 6, 2021
Copy link
Contributor

@bartoszgadomski bartoszgadomski left a comment

Choose a reason for hiding this comment

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

Looks good to me 👍

…support-request

* 'develop' of github.com:ampproject/amp-wp: (22 commits)
  Update Gutenberg package dependencies
  Run phpunit with verbose
  Skip SupportLink and SupportScreen tests when Site Health is not present
  Ensure support links are only added when user has access to support screen
  Add view_site_health_checks cap check requirement for SupportScreen
  Add tests for changes to SupportScreen
  Add tests for expected inline script data
  Add E2E tests to check if scanner is disabled if user has no capability
  Reduce `siteScanReducer` complexity
  Skip Site Scan if no validation nonce is provided
  Run ergebnis/composer-normalize
  Supply aspect-ratio to videos in Video Blocks
  Add height:auto to videos in Video blocks
  Use intrinsic layout for video instead of responsive
  Remove obsolete UserContextProvider from site-scan-notice after 473fdbd
  Omit updates nonce when user lacks install-themes cap anyway
  Limit access to validate_nonce to users who can amp_validate
  Limit access to SupportScreen to users who can amp_validate
  Add missing import for createInterpolateElement
  Improve translation strings
  ...
@westonruter westonruter enabled auto-merge December 7, 2021 17:14
@westonruter westonruter changed the title Use AMP_Validated_URL_Post_Type::get_url_from_post() instead of post_title property Use AMP_Validated_URL_Post_Type::get_url_from_post() instead of post_title property Dec 7, 2021
@westonruter westonruter merged commit 0ab628a into develop Dec 7, 2021
@westonruter westonruter deleted the bug/6707-support-request branch December 7, 2021 17:18
@westonruter westonruter added the Changelogged Whether the issue/PR has been added to release notes. label Dec 15, 2021
@delawski
Copy link
Collaborator

QA Passed

✅ The Validated URL is correct on the Support screen

Screenshot 2021-12-15 at 12 23 24

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changelogged Whether the issue/PR has been added to release notes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants