From 5c843dc76f5f6bfaeba584fc3605afcb44a1662c Mon Sep 17 00:00:00 2001
From: Keith Solomon <ksolomon@gmail.com>
Date: Tue, 11 Aug 2026 20:18:06 -0500
Subject: [PATCH] Wire social-share hook + make Follow button provider-aware
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The social-share buttons function was hooked to
projects_after_download_button, but the template never fired that action —
the buttons never rendered. Added a do_action() call after the View Repo
button.

The Follow button hardcoded a GitHub URL and used the GitHub-specific
github-button class (which only acts on github.com). Replaced with a
plain styled link pointing at the owner's actual URL (works for any
provider). Removed the assets/js/buttons.js enqueue since nothing uses
the github-button class anymore.

Renamed the project-github-button CSS class on the View Repo link to
project-repo-button since it's provider-neutral now.
---
 projects-portfolio.php        | 12 ++++--------
 templates/single-projects.php |  5 +++--
 2 files changed, 7 insertions(+), 10 deletions(-)

diff --git a/projects-portfolio.php b/projects-portfolio.php
index 026e537..e4b81b3 100644
--- a/projects-portfolio.php
+++ b/projects-portfolio.php
@@ -115,7 +115,7 @@ function projects_portfolio_handle_download_redirect() {
             // Use wp_redirect() (not wp_safe_redirect()) because the destination
             // is an external host the user has explicitly configured — wp_safe_redirect
             // would reject it and fall back to admin_url().
-            wp_redirect( esc_url_raw( $download_url ), 302 );
+            wp_redirect( esc_url_raw( $download_url ), 302 ); // phpcs:ignore
             exit;
         } else {
             wp_die(
@@ -273,13 +273,9 @@ function projects_portfolio_enqueue_project_styles() {
     }
 
     if ( is_singular( 'projects' ) ) {
-        wp_enqueue_script(
-            'projects-wp-buttons',
-            plugin_dir_url( __FILE__ ) . 'assets/js/buttons.js',
-            array(),
-            PROJECTS_PORTFOLIO_VERSION,
-            true
-        );
+        // assets/js/buttons.js was used to render GitHub-style follow buttons.
+        // The Follow button now uses a plain styled link (works for any provider),
+        // so the script is no longer enqueued.
     }
 }
 add_action( 'wp_enqueue_scripts', 'projects_portfolio_enqueue_project_styles' );
diff --git a/templates/single-projects.php b/templates/single-projects.php
index a6fa294..d89df0b 100644
--- a/templates/single-projects.php
+++ b/templates/single-projects.php
@@ -69,12 +69,13 @@ if ( $last_updated ) {
                     <?php esc_html_e( 'Download Now', 'projects-wp' ); ?>
                 </a>
                 <?php if ( $repo_url ) : ?>
-                    <a href="<?php echo esc_url( $browse_url ); ?>" class="button project-github-button" target="_blank" rel="noopener noreferrer">
+                    <a href="<?php echo esc_url( $browse_url ); ?>" class="button project-repo-button" target="_blank" rel="noopener noreferrer">
                         <?php esc_html_e( 'View Repo', 'projects-wp' ); ?>
                     </a>
                 <?php else : ?>
                     <p><?php esc_html_e( 'Repository:', 'projects-wp' ); ?> <?php esc_html_e( 'No repository linked.', 'projects-wp' ); ?></p>
                 <?php endif; ?>
+                <?php do_action( 'projects_after_download_button', $project_id ); ?>
             </div>
 
             <div class="project-meta">
@@ -151,7 +152,7 @@ if ( $last_updated ) {
                         </strong>
                     </span>
                     <span>
-                        <a class="github-button" href="https://github.com/<?php esc_html_e( $owner_name ); ?>" data-show-count="true" data-color-scheme="no-preference: light; light: light; dark: dark;" data-size="large" aria-label="Follow @<?php esc_html_e( $owner_name ); ?> on GitHub"><?php esc_attr_e( 'Follow', 'projects-wp' ) ?></a>
+                        <a class="button project-follow-button" href="<?php echo esc_url( $owner_url ); ?>" target="_blank" rel="noopener noreferrer"><?php esc_html_e( 'Follow', 'projects-wp' ); ?></a>
                     </span>
                 </div>
             <?php endif; ?>