plan_my_stuff 0.21.1 → 0.22.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4eda8634e75b318cd847d512405f71d4ffe2b0d7a6beccd6404912a6c64f1c2b
-  data.tar.gz: 80c7edb2a0e861c3dc0f76b06ae23232bde0ed52d04f4cdada69bd29aa8107dd
+  metadata.gz: d44fbaa2722319a0d586bf8f947222aa76f74462b5516f516f34b87b89ff6211
+  data.tar.gz: 05eaab50eaca5856a19cd68f796b1b4170cb750a6bb8c78566c003a576707122
 SHA512:
-  metadata.gz: 3336630d6d5ee6b8bad51fdfa08a647aa58933e0f30a36241fd137a4d3f5b67656386d36fd0990d289499a7cf2643469e32c1642d4f2d5811be5fc671ea9d420
-  data.tar.gz: 15db293bc2203d60ca7e2b5d102035b1a8d064a74b84666e13d0fbcad1ab15dbe576f02af2f7582c56c9f65246dbb23e41043f6036e304801722c40890c463fc
+  metadata.gz: afbbeca592468025940d1b98eae3c93da09ca79a5406afe5d500f93e8d1803c59d8ced137c81b1e05297678de17920681756983cd0b2a4c253900897f4d5d9b5
+  data.tar.gz: 95ca844953334f3d514c1b530cac5837307b3b735975c18256ab09ad408e1e4aea1f964de8d43ffe4c6b74bdf17284f93d8f114f7e47c4ecb37484023278b28e

data/CHANGELOG.md

@@ -1,5 +1,18 @@
 # Changelog
 
+## 0.22.0
+
+### Added
+
+- `Issue#add_to_priority_list!(priority:)` and `Issue#remove_from_priority_list!` - first-class setters that
+  mirror the `#priority_list?` / `#priority_list_priority` readers. Each writes both `Priority List` and
+  `Priority List Priority` in a single mutation so the two fields cannot drift apart (closes #80).
+
+### Changed
+
+- `IssueMetadata#priority_list` / `#priority_list_priority` deprecation message now points callers at the new
+  `Issue#add_to_priority_list!` / `#remove_from_priority_list!` pair instead of raw `set_issue_fields!`.
+
 ## 0.21.1
 
 ### Added

data/lib/plan_my_stuff/issue.rb

@@ -842,6 +842,35 @@ module PlanMyStuff
       issue_fields['Priority List Priority']
     end
 
+    # Adds this issue to the Priority List at the given priority (or re-prioritizes if already listed). Sets
+    # +Priority List+ and +Priority List Priority+ together in a single +setIssueFieldValue+ mutation so the two
+    # fields never drift out of sync.
+    #
+    # @raise [PlanMyStuff::IssueFieldsNotEnabledError] when +config.issue_fields_enabled+ is +false+
+    #
+    # @param priority [Integer]
+    #
+    # @return [self]
+    #
+    def add_to_priority_list!(priority:)
+      raise(ArgumentError, 'Priority must be an integer') unless priority.is_a?(Integer)
+
+      set_issue_fields!('Priority List' => 'Yes', 'Priority List Priority' => priority)
+    end
+
+    alias update_priority_list_priority! add_to_priority_list!
+
+    # Removes this issue from the Priority List. Clears both +Priority List+ and +Priority List Priority+ in a single
+    # +setIssueFieldValue+ mutation so the two fields never drift out of sync.
+    #
+    # @raise [PlanMyStuff::IssueFieldsNotEnabledError] when +config.issue_fields_enabled+ is +false+
+    #
+    # @return [self]
+    #
+    def remove_from_priority_list!
+      set_issue_fields!('Priority List' => nil, 'Priority List Priority' => nil)
+    end
+
     # Bulk-updates GitHub Issue Field values in a single +setIssueFieldValue+ mutation. Each key is the field display
     # name; values are coerced to the right input fragment based on the field's type. Passing +nil+ as a value clears
     # that field.

data/lib/plan_my_stuff/issue_metadata.rb

@@ -5,7 +5,8 @@ module PlanMyStuff
     PRIORITY_LIST_METADATA_DEPRECATION =
       'PlanMyStuff: IssueMetadata#priority_list / #priority_list_priority are deprecated. priority_list moved to ' \
         "GitHub Issue Fields ('Priority List' single_select, 'Priority List Priority' number) in 0.20.0. Reads " \
-        'now come from issue_fields; legacy writes forward to set_issue_fields! on save. The metadata accessors ' \
+        'now come from Issue#priority_list? / #priority_list_priority; for writes, use ' \
+        'Issue#add_to_priority_list!(priority: N) and Issue#remove_from_priority_list!. The metadata accessors ' \
         'will be removed in 1.0.0.'
 
     # @return [Time, nil] first support action timestamp, nil until set
@@ -201,7 +202,7 @@ module PlanMyStuff
       @priority_list
     end
 
-    # @deprecated Use +Issue#update!(issue_fields: { 'Priority List' => 'Yes' })+. Removed in 1.0.0.
+    # @deprecated Use +Issue#add_to_priority_list!(priority: N)+ / +Issue#remove_from_priority_list!+. Removed in 1.0.0.
     #
     # @param value [Object]
     #
@@ -231,7 +232,7 @@ module PlanMyStuff
       @priority_list_priority
     end
 
-    # @deprecated Use +Issue#update!(issue_fields: { 'Priority List Priority' => N })+. Removed in 1.0.0.
+    # @deprecated Use +Issue#add_to_priority_list!(priority: N)+ / +Issue#remove_from_priority_list!+. Removed in 1.0.0.
     #
     # @param value [Object]
     #

data/lib/plan_my_stuff/version.rb

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

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: plan_my_stuff
 version: !ruby/object:Gem::Version
-  version: 0.21.1
+  version: 0.22.0
 platform: ruby
 authors:
 - Brands Insurance