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

Fix AMP version comment pages URL for legacy paired URL structure. #6389

Merged
merged 21 commits into from Jun 17, 2021

Commits on Jun 15, 2021

  1. Configuration menu
    Copy the full SHA
    1b0623d View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    a2fdf96 View commit details
    Browse the repository at this point in the history
  3. Update unit test cases

    dhaval-parekh committed Jun 15, 2021
    Configuration menu
    Copy the full SHA
    9a91e7d View commit details
    Browse the repository at this point in the history
  4. Fix php linting issue

    dhaval-parekh committed Jun 15, 2021
    Configuration menu
    Copy the full SHA
    936314b View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    324dd5f View commit details
    Browse the repository at this point in the history

Commits on Jun 16, 2021

  1. Configuration menu
    Copy the full SHA
    604ef89 View commit details
    Browse the repository at this point in the history
  2. Simplify regex given that amp_get_slug() always returns query var

    Co-authored-by: Pierre Gordon <16200219+pierlon@users.noreply.github.com>
    westonruter and pierlon committed Jun 16, 2021
    Configuration menu
    Copy the full SHA
    eaa064f View commit details
    Browse the repository at this point in the history
  3. Removed needless fallback for AMP slug

    Co-authored-by: Pierre Gordon <16200219+pierlon@users.noreply.github.com>
    westonruter and pierlon committed Jun 16, 2021
    Configuration menu
    Copy the full SHA
    5a3dfa7 View commit details
    Browse the repository at this point in the history
  4. Remove extra newline

    Co-authored-by: Pierre Gordon <16200219+pierlon@users.noreply.github.com>
    westonruter and pierlon committed Jun 16, 2021
    Configuration menu
    Copy the full SHA
    b6db697 View commit details
    Browse the repository at this point in the history
  5. Remove extraneous comment

    Co-authored-by: Pierre Gordon <16200219+pierlon@users.noreply.github.com>
    westonruter and pierlon committed Jun 16, 2021
    Configuration menu
    Copy the full SHA
    0bbfbc0 View commit details
    Browse the repository at this point in the history
  6. Configuration menu
    Copy the full SHA
    d769474 View commit details
    Browse the repository at this point in the history

Commits on Jun 17, 2021

  1. Configuration menu
    Copy the full SHA
    f387fc2 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    acf8a17 View commit details
    Browse the repository at this point in the history
  3. Rename method name

    westonruter committed Jun 17, 2021
    Configuration menu
    Copy the full SHA
    2115192 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    d9a481d View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    c70460e View commit details
    Browse the repository at this point in the history
  6. Merge branch 'develop' of github.com:ampproject/amp-wp into bug/6355-…

    …legacy-reader-paired-url
    
    * 'develop' of github.com:ampproject/amp-wp: (56 commits)
      Fix path regex for QA plugin workflow
      Fix script path
      Fix syntax error
      Also apply modified paths restrictions for QA tester plugin workflow
      Fix patterns
      Only run jobs when related files are modified
      Rename step
      Update Gutenberg package dependencies
      Omit Bento from components when version is not valid
      Update spec to include ampproject/amphtml#34769
      Update Gutenberg package dependencies
      Set null state
      Require polyfill directly
      Replace deprecated `@babel/polyfill` with `@wordpress/babel-preset-default/build/polyfill`
      Parse result into array
      Close latest Gutenberg dependency update PR if it is outdated
      Enable auto merge if needed
      Remove dependency approve list
      Switch to `google-github-actions/setup-gcloud` action
      Fix lint error
      ...
    westonruter committed Jun 17, 2021
    Configuration menu
    Copy the full SHA
    72408af View commit details
    Browse the repository at this point in the history
  7. Configuration menu
    Copy the full SHA
    fad355a View commit details
    Browse the repository at this point in the history
  8. Run filter_redirect_canonical_to_fix_cpage_requests earlier so other …

    …redirect_canonical filters have fixed URL
    westonruter committed Jun 17, 2021
    Configuration menu
    Copy the full SHA
    9a0fc1a View commit details
    Browse the repository at this point in the history
  9. Merge branch 'develop' of github.com:ampproject/amp-wp into bug/6355-…

    …legacy-reader-paired-url
    
    * 'develop' of github.com:ampproject/amp-wp:
      Bump grunt from 1.4.0 to 1.4.1 in /qa-tester
      Bump codecov/codecov-action from 1.5.0 to 1.5.2
      Bump cweagans/composer-patches from 1.7.0 to 1.7.1
      Bump @wordpress/eslint-plugin from 9.0.5 to 9.0.6 in /qa-tester
      Bump eslint from 7.26.0 to 7.28.0
      Bump grunt from 1.4.0 to 1.4.1
      Bump php-stubs/wordpress-stubs from 5.7.1 to 5.7.2
      Bump google/cloud-storage from 1.23.1 to 1.23.2
      Bump postcss-loader from 4.2.0 to 4.3.0
    westonruter committed Jun 17, 2021
    Configuration menu
    Copy the full SHA
    129f693 View commit details
    Browse the repository at this point in the history
  10. Configuration menu
    Copy the full SHA
    28b3fbf View commit details
    Browse the repository at this point in the history