From 34c91bbc9b1155336a4289d701952312641ebc5f Mon Sep 17 00:00:00 2001
From: Corey McKrill <916023+coreymckrill@users.noreply.github.com>
Date: Fri, 4 Oct 2019 12:22:26 -0700
Subject: [PATCH] Add some comments

---
 plugins/wporg-5ftf/includes/pledge-meta.php     | 2 ++
 plugins/wporg-5ftf/includes/shortcodes.php      | 2 ++
 plugins/wporg-5ftf/views/form-pledge-manage.php | 2 +-
 plugins/wporg-5ftf/views/front-end.php          | 6 ++++--
 4 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/plugins/wporg-5ftf/includes/pledge-meta.php b/plugins/wporg-5ftf/includes/pledge-meta.php
index 5713fb0..49f0bfd 100755
--- a/plugins/wporg-5ftf/includes/pledge-meta.php
+++ b/plugins/wporg-5ftf/includes/pledge-meta.php
@@ -196,6 +196,8 @@ function has_required_pledge_meta( array $submission ) {
 /**
  * Save the pledge data.
  *
+ * This only fires when the pledge post itself is created or updated.
+ *
  * @param int     $pledge_id
  * @param WP_Post $pledge
  */
diff --git a/plugins/wporg-5ftf/includes/shortcodes.php b/plugins/wporg-5ftf/includes/shortcodes.php
index ada8a10..5a4b36e 100755
--- a/plugins/wporg-5ftf/includes/shortcodes.php
+++ b/plugins/wporg-5ftf/includes/shortcodes.php
@@ -5,6 +5,8 @@
  * or two, we may want to create a subfolder and have a separate file for each block.
  */
 
+// TODO are we actually using any of this?
+
 namespace WordPressDotOrg\FiveForTheFuture\Blocks;
 use WordPressDotOrg\FiveForTheFuture;
 use WordPressDotOrg\FiveForTheFuture\Pledge;
diff --git a/plugins/wporg-5ftf/views/form-pledge-manage.php b/plugins/wporg-5ftf/views/form-pledge-manage.php
index bce888f..b014cc6 100644
--- a/plugins/wporg-5ftf/views/form-pledge-manage.php
+++ b/plugins/wporg-5ftf/views/form-pledge-manage.php
@@ -7,7 +7,7 @@ use function WordPressDotOrg\FiveForTheFuture\get_views_path;
 /** @var bool  $updated */
 ?>
 
-<form id="5ftf-form-pledge-new" action="" method="post">
+<form id="5ftf-form-pledge-manage" action="" method="post">
 	<?php
 	require get_views_path() . 'inputs-pledge-org-info.php';
 	require get_views_path() . 'inputs-pledge-org-logo.php';
diff --git a/plugins/wporg-5ftf/views/front-end.php b/plugins/wporg-5ftf/views/front-end.php
index 6b72fd9..4e1696f 100755
--- a/plugins/wporg-5ftf/views/front-end.php
+++ b/plugins/wporg-5ftf/views/front-end.php
@@ -1,6 +1,8 @@
 <?php // todo i18n
 //// change all id/class prefixes to fftf (or something better) b/c not valid to start w/ number
-///  ?>
+
+// TODO are we using this, or is all the front end stuff happening in the 5ftF theme now?
+?>
 
 <article class="5ftf">
 	<section class="about">
@@ -96,4 +98,4 @@
 
 		<?php // link to pledge form ?>
 	</section>
-</article>
\ No newline at end of file
+</article>