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

Rename AMP Invalid URLs to AMP Validated URLs #1487

Merged
merged 11 commits into from Oct 4, 2018
Merged

Conversation

@kienstra
Copy link
Contributor

@kienstra kienstra commented Oct 4, 2018

  • Changes the labels on the post type index page and the 'AMP Validation Error Index.'
  • Updates comments that reference that page from 'Invalid URLs' to 'AMP Validation URLs'
  • Updates post type slug from amp_invalid_url to amp_validated_url.
  • Updates class name from AMP_Invalid_URL to AMP_Validated_URL.
  • Updates filenames and other references with s/invalud/validated/

Before

before-invalid-urls

After

after-invalid-urls

after-error-index

kienstra and others added 5 commits Oct 4, 2018
The name is now 'AMP Validated URLs',
and the menu name is 'Validated URLs.'
But this does not change the slug,
as that probably belongs in a separate PR
that replaces the post_type of existing posts.
Only for AMP_Validation_Error_Taxonomy,
as the previous commit applied to
AMP_Invalid_URL_Post_Type.
@westonruter westonruter changed the title Change validation post type labels Rename AMP Invalid URLs to AMP Validated URLs Oct 4, 2018
amp.php Show resolved Hide resolved
Loading
amedina
amedina approved these changes Oct 4, 2018
Copy link
Member

@amedina amedina left a comment

Tested the branch and everything is in place. LGTM!

Loading

@westonruter westonruter added this to the v1.0 milestone Oct 4, 2018
@kienstra
Copy link
Contributor Author

@kienstra kienstra commented Oct 4, 2018

Approved

Hi @westonruter,
This looks good.

On pulling checking out this branch and loading a page, the post types are updated as expected.

Before

before-change-post-type

After

after-counts-appear-same

Also, the version is stored in an option, as expected:
version-stored

Loading

@westonruter westonruter merged commit 5ce3414 into 1.0 Oct 4, 2018
2 checks passed
Loading
@westonruter westonruter deleted the update/post-type-label branch Oct 4, 2018
@kienstra
Copy link
Contributor Author

@kienstra kienstra commented Oct 5, 2018

Looks Good On Staging

Getting ready for the RC1 release, I uploaded the built amp.zip to some staging sites.

The new logic looks to have converted the post types well.

https://paired-ampconfdemo.pantheonsite.io
new-post-type

https://native-ampconfdemo.pantheonsite.io
native-post-type

Loading

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants