WordPress.org

Making WordPress.org

Opened 2 months ago

Closed 2 months ago

Last modified 2 months ago

#5607 closed defect (fixed)

Remove ?view=all param from profile thread responses

Reported by: jonoaldersonwp Owned by: dd32
Milestone: Priority: lowest
Component: Support Forums Keywords: seo performance
Cc:

Description

When a user comments on a thread, it's recorded in their profile.
The link to that thread appends a ?view=all parameter, which is redundant. This should be removed (in existing, and new response records).

Change History (7)

#1 @fierevere
2 months ago

It still can be a good idea to automatically append "?view=all" for moderators.

Last edited 2 months ago by fierevere (previous) (diff)

#2 @Otto42
2 months ago

Where is this view=all at? Because I don't see it in any links on profiles.

#3 @jonoaldersonwp
2 months ago

Most recent reply on https://profiles.wordpress.org/sterndata/, for example.

Posted a <a href="https://wordpress.org/support/topic/database-error-322/?view=all#post-14033241">reply</a> to <i>Database Error</i>, on the site WordPress.org Forums:

#4 @Otto42
2 months ago

Yeah, that's not universal.

I think that is only happening on moderator's replies, when they are using the view=all view that lets them see the archived replies as well.

If this is undesirable, then the correct thing to do would be to find the piece of the forums that sends the reply notification to the profiles and remove the data from it there, so that it's sending the desired URL.

#5 @dd32
2 months ago

  • Component changed from Profiles to Support Forums

#6 @dd32
2 months ago

  • Owner set to dd32
  • Resolution set to fixed
  • Status changed from new to closed

In 10663:

Support: Don't include moderator query vars in the permalink sent to the Profiles feed.

Fixes #5607.

#7 @dd32
2 months ago

Note that the URL sent over by [10663] will now be .../topic-name/?#post-1234 with the extra ? in there, that's due to #core52503.

Note: See TracTickets for help on using tickets.