search

Automattic / WP-Job-Manager

Manage job listings from the WordPress admin panel, and allow users to post jobs directly to your site.
https://wpjobmanager.com
GNU General Public License v3.0
895 stars 366 forks source link

Optimize job listing view count #2732

Closed fjorgemota closed 4 months ago

fjorgemota commented 5 months ago

Follow-up to #2728

Changes Proposed in this Pull Request

Testing Instructions

On a WP installation running this version of the plugin:

  1. Create a job listing
  2. Visit the job listing
  3. Verify if the count in the table wp_wpjm_stats is increased
  4. Visit the job listing again
  5. Verify if the count in the table wp_wpjm_stats is increased

Release Notes

Plugin build for ea947e153f4547c6bdf5dc914852e55c87e1b0a2
📦 Download plugin zip
▶️ Open in playground
pgk commented 4 months ago

In light of the more AJAX-driven approach this feature (stats recording) is taking, do you think we can skip this optimization at this point?

fjorgemota commented 4 months ago

Hey, yeah. Sure. I'm going to close this issue as it is clearly not needed anymore.

Thanks for the heads-up anyway!