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.

Sign up for GitHub

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

Jump to bottom

Add mechanism to report comment content modifications via sync #6929

Merged
merged 12 commits into from
Apr 6, 2017
Merged

Add mechanism to report comment content modifications via sync #6929

merged 12 commits into from
Apr 6, 2017

Commits on Apr 5, 2017

  1. Jetpack Sync: Adding listener for edit_comment action so Activity Log… 

    … knows when comment was edited
    @gititon
    gititon committed Apr 5, 2017
    Configuration menu
    Copy the full SHA
    856c60a View commit details
    Browse the repository at this point in the history

  2. Merge branch 'master' of github.com:Automattic/jetpack into add/comme… 

    …nt_sync_status
    @gititon
    gititon committed Apr 5, 2017
    Configuration menu
    Copy the full SHA
    0fa60d6 View commit details
    Browse the repository at this point in the history

  3. Jetpack Sync: Adding test to ensure edit_comment action is synced whe… 

    …n comment contents is updated
    @gititon
    gititon committed Apr 5, 2017
    Configuration menu
    Copy the full SHA
    4c0554b View commit details
    Browse the repository at this point in the history

  4. Jetpack Sync: Adding listener for wp_update_comment_data 

    Adding listener for wp_update_comment_data so I can compare old comment contents with new comment contents and conditionally invoke another action that the sync listener will listen for
    @gititon
    gititon committed Apr 5, 2017
    Configuration menu
    Copy the full SHA
    2f4b925 View commit details
    Browse the repository at this point in the history

  5. Jetpack Sync: Changing add_action to add_filter to invoke expected be… 

    …havior
    @gititon
    gititon committed Apr 5, 2017
    Configuration menu
    Copy the full SHA
    1379bf8 View commit details
    Browse the repository at this point in the history

  6. Jetpack Sync: Adding debugging

    @gititon
    gititon committed Apr 5, 2017
    Configuration menu
    Copy the full SHA
    4f5f53d View commit details
    Browse the repository at this point in the history

  7. Jetpack Sync: Invoking action if comment comments are modified that s… 

    …ync listeners are listening for
    @gititon
    gititon committed Apr 5, 2017
    Configuration menu
    Copy the full SHA
    244208b View commit details
    Browse the repository at this point in the history

  8. Jetpack Sync: Updating name of sync action tested for in Comment Modu… 

    …le test
    @gititon
    gititon committed Apr 5, 2017
    Configuration menu
    Copy the full SHA
    d4fdea5 View commit details
    Browse the repository at this point in the history

  9. Jetpack Sync: Updated comment modified mechanism to account for and t… 

    …est author, author email, author url update
    @gititon
    gititon committed Apr 5, 2017
    Configuration menu
    Copy the full SHA
    e39239f View commit details
    Browse the repository at this point in the history

  10. Jetpack Sync: Removing real email address from test data

    @gititon
    gititon committed Apr 5, 2017
    Configuration menu
    Copy the full SHA
    5ed88a2 View commit details
    Browse the repository at this point in the history

  11. Jetpack Sync: Suggested code review changes

    @gititon
    gititon committed Apr 5, 2017
    Configuration menu
    Copy the full SHA
    e088ef5 View commit details
    Browse the repository at this point in the history

Commits on Apr 6, 2017

  1. Jetpack Sync: Skipping comment content modified tests if WP version <… 

    … 4.7 since it lacks wp_update_comment_data filter
    @gititon
    gititon committed Apr 6, 2017
    Configuration menu
    Copy the full SHA
    8019ba3 View commit details
    Browse the repository at this point in the history