plan_my_stuff 0.28.0 → 0.29.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9aac5ec777cf9b53a459843e9bdbf1177178fe1623457a60d266e993db3d2d93
-  data.tar.gz: 1b7089b760fa0552e3b861732472e43cab09667699e1528703809a2ce753254f
+  metadata.gz: 98cbaeb499aa1b3b22018b7145d2d8e0bd86051a237230d1b5cac3e884109e10
+  data.tar.gz: a66aa1aec786adca0853eed340e1f35e2639c7aca6aafae88a421b4cc2136b5d
 SHA512:
-  metadata.gz: 84f1cc4e7dd87ed3e33f2095d3c0f57c9ad61972ffaa5ed1d9379dfb6bb0e7bc1b07491c76c6c4f4618c39b977d7240689de5552a8c9f60fe1b474c29dde2637
-  data.tar.gz: 1d355a051c6d6f0903c80558019790ab0e1d84adb8885b28b0c18b57802ed16c338ab7793ca24f535dc01980296fcc33b19eacc3acd204d08586db25d63138d4
+  metadata.gz: 8f4e1995d828ae83edfc94a9d2d0805b5787165e28f062bd0100f586de2c19e49fb6f9167ac0ee0ea88be5001012ffab5371315d2dcaee4b081a13e6be53a993
+  data.tar.gz: 95474b37f6f2bbd4c6074f3049086b1d643617c7c0a6dbf8562e9e0c29f9ad7bcbcabcedfda29f8e55cfcf12d5f0024a41cd69d0c9527dff91948a83e8dcde46

data/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## 0.29.0
+
+### Changed
+
+- `Pipeline.complete_deployment!` now sets the linked issue's `Issue Status` field to `Fixed` on the auto-complete
+  success path (alongside moving the item to `Completed`). Gated on `config.issue_fields_enabled`, so completion still
+  succeeds when issue fields are disabled. The auto-complete-off early return is unchanged (closes #96).
+
 ## 0.28.0
 
 ### Added

data/app/controllers/plan_my_stuff/issues/approvals_controller.rb

@@ -23,10 +23,11 @@ module PlanMyStuff
           return
         end
 
-        issue = PlanMyStuff::Issue.find(params[:issue_id])
-        issue.request_approvals!(user_ids: user_ids, user: pms_current_user)
+        find_issue
 
-        yield(issue) if block_given?
+        @issue.request_approvals!(user_ids: user_ids, user: pms_current_user)
+
+        yield(@issue) if block_given?
         return if performed?
 
         flash[:success] = 'Approvers were successfully added.'
@@ -47,7 +48,7 @@ module PlanMyStuff
           return
         end
 
-        issue = PlanMyStuff::Issue.find(params[:issue_id])
+        find_issue
         caller_id = pms_current_user.present? ? PlanMyStuff::UserResolver.user_id(pms_current_user) : nil
 
         case status
@@ -56,25 +57,25 @@ module PlanMyStuff
             redirect_to_unauthorized(show_path)
             return
           end
-          issue.approve!(user: pms_current_user)
+          @issue.approve!(user: pms_current_user)
           flash[:success] = 'Approval recorded.'
         when 'rejected'
           unless caller_id == target_id
             redirect_to_unauthorized(show_path)
             return
           end
-          issue.reject!(user: pms_current_user)
+          @issue.reject!(user: pms_current_user)
           flash[:success] = 'Rejection recorded.'
         else
           if caller_id != target_id && !support_user?
             redirect_to_unauthorized(show_path)
             return
           end
-          issue.revoke_approval!(user: pms_current_user, target_user_id: target_id)
+          @issue.revoke_approval!(user: pms_current_user, target_user_id: target_id)
           flash[:success] = 'Response revoked.'
         end
 
-        yield(issue) if block_given?
+        yield(@issue) if block_given?
         return if performed?
 
         redirect_to(show_path)
@@ -91,10 +92,10 @@ module PlanMyStuff
           return
         end
 
-        issue = PlanMyStuff::Issue.find(params[:issue_id])
-        issue.remove_approvers!(user_ids: [params[:id].to_i], user: pms_current_user)
+        find_issue
+        @issue.remove_approvers!(user_ids: [params[:id].to_i], user: pms_current_user)
 
-        yield(issue) if block_given?
+        yield(@issue) if block_given?
         return if performed?
 
         flash[:success] = 'Approver was successfully removed.'
@@ -112,9 +113,14 @@ module PlanMyStuff
           params.fetch(:approval, {}).permit(:status, :user_ids)
         end
 
+        # :nodoc:
+        def find_issue
+          @issue = PlanMyStuff::Issue.find(params[:issue_id])
+        end
+
         # @return [String]
         def show_path
-          plan_my_stuff.issue_path(params[:issue_id])
+          plan_my_stuff.issue_path(@issue || params[:issue_id])
         end
     end
   end

data/lib/plan_my_stuff/engine.rb

@@ -16,7 +16,7 @@ module PlanMyStuff
       if loader.respond_to?(:eager_load_dir)
         loader.eager_load_dir(controllers_dir)
       else
-        Dir.glob(File.join(controllers_dir, '**/*.rb')).sort.each { |path| require path }
+        Dir.glob(File.join(controllers_dir, '**/*.rb')).each { |path| require path }
       end
     end
   end

data/lib/plan_my_stuff/pipeline.rb

@@ -273,8 +273,10 @@ module PlanMyStuff
       items
     end
 
-    # Moves a project item to "Completed" if the linked issue has +auto_complete+ enabled. Returns +nil+ when
-    # auto-complete is off (item stays at "Release in Progress").
+    # Moves a project item to "Completed" if the linked issue has +auto_complete+ enabled, and sets the linked issue's
+    # "Issue Status" field to "Fixed". Returns +nil+ when auto-complete is off (item stays at "Release in Progress" and
+    # no field is touched). The "Issue Status" update is skipped when +config.issue_fields_enabled+ is +false+ so
+    # completion still succeeds.
     #
     # @param project_item [PlanMyStuff::ProjectItem]
     # @param deployment_id [String, nil]
@@ -288,6 +290,8 @@ module PlanMyStuff
       status = resolve_status_name(PlanMyStuff::Pipeline::Status::COMPLETED)
       result = project_item.move_to!(status)
 
+      issue.set_issue_fields!('Issue Status' => 'Fixed') if PlanMyStuff.configuration.issue_fields_enabled
+
       instrument(PlanMyStuff::Pipeline::Status::COMPLETED, project_item, deployment_id: deployment_id)
       result
     end

data/lib/plan_my_stuff/version.rb

@@ -3,7 +3,7 @@
 module PlanMyStuff
   module VERSION
     MAJOR = 0
-    MINOR = 28
+    MINOR = 29
     TINY = 0
 
     # Set PRE to nil unless it's a pre-release (beta, rc, etc.)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: plan_my_stuff
 version: !ruby/object:Gem::Version
-  version: 0.28.0
+  version: 0.29.0
 platform: ruby
 authors:
 - Brands Insurance
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-05-26 00:00:00.000000000 Z
+date: 2026-05-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails