WordCamp activity in Profiles shouldn't say "coming up on [date]" for past dates

[valentinbora]

Example:

• Joined the organizing team for WordCamp Jackson, Michigan 2019 coming up on October 4th
• Confirmed as a speaker for WordCamp Dayton, OH, USA 2019 coming up on March 1st

These are for last year's WordCamps but the activity says "coming up on [date]", and it doesn't include a year. This can be potentially misleading.

[dd32]

The way BuddyPress Activity feeds works is that these are static strings stored in the database, they're not dynamic (although could probably be filtered at display)

Perhaps we should change the strings to just

• Joined the organizing team for WordCamp Jackson, Michigan 2019
• Confirmed as a speaker for WordCamp Dayton, OH, USA 2019

[valentinbora]

@dd32 makes sense, I feared as much. I appreciate your suggestions.

[valentinbora]

@dd32 could we instead change sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-participation-notifier/wordcamp-participation-notifier.php#L360 to provide the full year and maybe even pass along another parameter with the actual timestamp?

[dd32]

Replying to valentinbora:

@dd32 could we instead change sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-participation-notifier/wordcamp-participation-notifier.php#L360 to provide the full year and maybe even pass along another parameter with the actual timestamp?

Oh good the WordCamp side of that is open source :)

Yes, we could pass the full date through, but I was questioning whether including the year/date of the event is remotely useful there, especially with the present/past/future tense issues.

I don't think the date of the event is the important information here, rather it's the action & Event name/link.

[valentinbora]

Remove "coming up on" verbiage for WordCamp activities

[valentinbora]

@dd32 apparently not all of it is open source, so I'd love to fix history on the fly through an adaptation of the following:

add_filter( 'bp_activity_get', function( $payload ) {
        if ( !empty($payload['activities'] ) && is_array( $payload['activities'] ) ) {
                foreach ( $payload['activities'] as &$activity ) {
                        if ( 'wordcamp' === $activity->component ) {
                                $activity->action = substr( $activity->action, 0, strpos( $activity->action, 'coming up on' ) );
                        }
                }
        }

        return $payload;
});

[dd32]

Replying to valentinbora:

@dd32 apparently not all of it is open source, so I'd love to fix history on the fly through an adaptation of the following:

Not all of profiles.w.org is open-source, Only the activity handler I think.

Rather than using a display filter, I think I'd probably alter the existing DB records directly to remove the unnecessary text.

I'm happy to make this change if we can get some buy-in from a community team member (Just to verify they don't feel that it's an important detail to keep displayed)
edit: ​Followed up with #community-team on Slack

[casiepa]

From my side I would say: remove that last part. As there is the link to the WordCamp site, there is no need for the date (what if it suddenly changes or gets canceled?)

[valentinbora]

@casiepa, @sippis thanks for chiming in.

@dd32 looks like we have two buy-in confirmations, good enough to proceed?

[SamSuresh]

I think it make sense to remove the date since the year is mentioned.

[dd32]

In 9508:

Profiles: Remove the WordCamp date from the Profiles activity feed.

This keeps the WordCamp name and year, but removes the "Coming up on Feb 30th" verbiage which is incorrect for all previous events.

Props valentinbora.
Fixes #5030.

[dd32]

Existing activity items updated too.