bloomy 0.4.2 → 0.5.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 453b980c92f83f6acf3d9a1191dcf94484a9a3acbc57811509f534964d0d15d7
-  data.tar.gz: 2c1e08c2351256c4d16b37729ce8173f144ed3680cbdd2e46f55c2a7e57b23f7
+  metadata.gz: 5967a74a2cb5758a74e4877ab46a57eeba59f85a023039c1252d47087007bfd7
+  data.tar.gz: 6ded65188a434b4a3fa47cb9007b5c1e10e97b88266c6442e4e74684b7f7f552
 SHA512:
-  metadata.gz: 2ae681b285f824f3ee6c5f5a9806a205d86d2c01fefe84263aa0b1d1d34bd1a3996b17dc98330051125a1b79d0f9e868ed79d7dc015294ba39a5fb71e3032ede
-  data.tar.gz: 9a7710243dc62d7bb21e6b1fb0f2795c44dbb034b0fab8f787a5b3d93c942be7ff9d25ed9bde6552e10859bfd52565d090d8b0d3f576dda31afe4da440a88707
+  metadata.gz: d5eaeb394ba3edbd99e433ab9760f510a9a0e219c5b0de8cf2fe3ef0cac7a03a12fbf3325515e3485629186d600724e7e8001d32070baf400a23b9453fd64476
+  data.tar.gz: 9298429c7361153eb4f58e35604defb752362add6f43bb253413c21abe11e2ae4f6d4997c03df337e2f12dcc8097685333ab68c7f0b12656b8f3594804f773fa

data/.rubocop.yml

@@ -1,30 +1,10 @@
-AllCops:
-  NewCops: enable
-  SuggestExtensions: false
-
-Gemspec/RequiredRubyVersion:
-  Enabled: false
-
-Metrics/AbcSize:
-  Enabled: false
-
-Metrics/BlockLength:
-  Enabled: false
-
-Metrics/CyclomaticComplexity:
-  Enabled: false
-
-Metrics/MethodLength:
-  Enabled: false
-
-Metrics/PerceivedComplexity:
-  Enabled: false
-
-Naming/AccessorMethodName:
-  Enabled: false
-
-Gemspec/DevelopmentDependencies:
-  Enabled: false
-
-Style/StringLiterals:
-  EnforcedStyle: double_quotes
+require:
+  - standard
+  - standard-custom
+  - standard-performance
+  - rubocop-performance
+
+inherit_gem:
+  standard: config/base.yml
+  standard-custom: config/base.yml
+  standard-performance: config/base.yml

data/lib/bloomy/client.rb

@@ -25,6 +25,9 @@ module Bloomy
     def initialize(api_key = nil)
       @configuration = Configuration.new unless api_key
       @api_key = api_key || @configuration.api_key
+
+      raise ArgumentError, "No API key provided. Set it in configuration or pass it directly." unless @api_key
+
       @base_url = "https://app.bloomgrowth.com/api/v1"
       @conn = Faraday.new(url: @base_url) do |faraday|
         faraday.response :json

data/lib/bloomy/operations/goals.rb

@@ -29,7 +29,7 @@ class Goal
         due_date: goal["DueDate"],
         status: goal["Complete"] ? "Completed" : "Incomplete",
         meeting_id: goal["Origins"].empty? ? nil : goal["Origins"][0]["Id"],
-        meeting_name: goal["Origins"].empty? ? nil : goal["Origins"][0]["Name"]
+        meeting_title: goal["Origins"].empty? ? nil : goal["Origins"][0]["Name"]
       }
     end
 
@@ -52,7 +52,7 @@ class Goal
       goal_id: response["Id"],
       title: title,
       meeting_id: meeting_id,
-      meeting_name: response["Origins"][0]["Name"],
+      meeting_title: response["Origins"][0]["Name"],
       user_id: user_id,
       user_name: response["Owner"]["Name"],
       created_at: DateTime.parse(response["CreateTime"])
@@ -68,7 +68,7 @@ class Goal
   #   #=> { status: 200 }
   def delete(goal_id)
     response = @conn.delete("/api/v1/rocks/#{goal_id}")
-    {status: response.status}
+    response.success?
   end
 
   # Updates a goal
@@ -83,7 +83,7 @@ class Goal
   def update(goal_id:, title:, accountable_user: user_id)
     payload = {title: title, accountableUserId: accountable_user}.to_json
     response = @conn.put("/api/v1/rocks/#{goal_id}", payload)
-    {status: response.status}
+    response.success?
   end
 
   private

data/lib/bloomy/operations/headlines.rb

@@ -56,7 +56,7 @@ class Headline
       notes_url: response.body["DetailsUrl"],
       meeting_details: {
         id: response.body["OriginId"],
-        name: response.body["Origin"]
+        title: response.body["Origin"]
       },
       owner_details: {
         id: response.body["Owner"]["Id"],
@@ -79,9 +79,7 @@ class Headline
   #  client.headline.list
   #  #=> [{ id: 1, title: "Headline Title", meeting_details: { id: 1, name: "Team Meeting" }, ... }, ...]
   def list(user_id: nil, meeting_id: nil)
-    if user_id && meeting_id
-      raise ArgumentError, "Please provide either `user_id` or `meeting_id`, not both."
-    end
+    raise ArgumentError, "Please provide either `user_id` or `meeting_id`, not both." if user_id && meeting_id
 
     if meeting_id
       response = @conn.get("/api/v1/l10/#{meeting_id}/headlines")
@@ -98,7 +96,7 @@ class Headline
         title: headline["Name"],
         meeting_details: {
           id: headline["OriginId"],
-          name: headline["Origin"]
+          title: headline["Origin"]
         },
         owner_details: {
           id: headline["Owner"]["Id"],
@@ -118,9 +116,6 @@ class Headline
   # @return [Boolean] true if the deletion was successful
   def delete(meeting_id, headline_id)
     response = @conn.delete("/api/v1/L10/#{meeting_id}/headlines/#{headline_id}")
-
-    # Raise an issue if response.status != 200
-    raise "Failed to delete headline" unless response.status == 200
-    true
+    response.success?
   end
 end

data/lib/bloomy/operations/issues.rb

@@ -30,7 +30,7 @@ class Issue
       completed_at: response["CloseTime"],
       meeting_details: {
         id: response["OriginId"],
-        name: response["Origin"]
+        title: response["Origin"]
       },
       owner_details: {
         id: response["Owner"]["Id"],
@@ -73,7 +73,7 @@ class Issue
         notes_url: issue["DetailsUrl"],
         created_at: issue["CreateTime"],
         meeting_id: issue["OriginId"],
-        meeting_name: issue["Origin"]
+        meeting_title: issue["Origin"]
       }
     end
   end
@@ -86,8 +86,8 @@ class Issue
   #   issue.complete(123)
   #   #=> true
   def complete(issue_id)
-    response = @conn.post("issues/#{issue_id}/complete", {complete: true}.to_json).status
-    response == 200
+    response = @conn.post("issues/#{issue_id}/complete", {complete: true}.to_json)
+    response.success?
   end
 
   # Creates a new issue

data/lib/bloomy/operations/meetings.rb

@@ -100,7 +100,7 @@ class Meeting
     response.map do |measurable|
       {
         id: measurable["Id"],
-        name: measurable["Name"].strip,
+        title: measurable["Name"].strip,
         target: measurable["Target"],
         operator: measurable["Direction"],
         format: measurable["Modifiers"],
@@ -132,7 +132,7 @@ class Meeting
     measurables = metrics(meeting_id)
     {
       id: meeting[:id],
-      name: meeting[:name],
+      title: meeting[:name],
       attendees: attendees,
       issues: issues,
       todos: todos,
@@ -166,6 +166,7 @@ class Meeting
   # @example
   #   client.meeting.delete(1)
   def delete(meeting_id)
-    @conn.delete("L10/#{meeting_id}")
+    response = @conn.delete("L10/#{meeting_id}")
+    response.success?
   end
 end

data/lib/bloomy/operations/scorecard.rb

@@ -49,10 +49,12 @@ class Scorecard
   #
   #   # Fetch scorecards for a specific meeting
   #   client.scorecard.list(meeting_id: 99)
+  # @note
+  #  The `week_offset` parameter is useful when fetching scores for previous or future weeks.
+  #  For example, to fetch scores for the previous week, you can set `week_offset` to -1.
+  #  To fetch scores for a future week, you can set `week_offset` to a positive value.
   def list(user_id: nil, meeting_id: nil, show_empty: false, week_offset: nil)
-    if user_id && meeting_id
-      raise ArgumentError, "Please provide either `user_id` or `meeting_id`, not both."
-    end
+    raise ArgumentError, "Please provide either `user_id` or `meeting_id`, not both." if user_id && meeting_id
 
     if meeting_id
       response = @conn.get("scorecard/meeting/#{meeting_id}").body
@@ -70,14 +72,15 @@ class Scorecard
         target: scorecard["Target"],
         value: scorecard["Measured"],
         week: scorecard["Week"],
+        week_id: scorecard["ForWeek"],
         updated_at: scorecard["DateEntered"]
       }
     end
 
     if week_offset
       week_data = current_week
-      week_id = week_data[:week_number] - week_offset
-      scorecards.select! { |scorecard| scorecard[:week] == week_id }
+      week_id = week_data[:week_number] + week_offset
+      scorecards.select! { |scorecard| scorecard[:week_id] == week_id }
     end
 
     scorecards.select! { |scorecard| scorecard[:value] || show_empty } unless show_empty
@@ -95,11 +98,11 @@ class Scorecard
   #  #=> true
   # @note
   #  The `week_offset` parameter is useful when updating scores for previous weeks.
-  #  For example, to update the score for the previous week, you can set `week_offset` to 1.
-  #  To update a future week's score, you can set `week_offset` to a negative value.
+  #  For example, to update the score for the previous week, you can set `week_offset` to -1.
+  #  To update a future week's score, you can set `week_offset` to a positive value.
   def score(measurable_id:, score:, week_offset: 0)
     week_data = current_week
-    week_id = week_data[:week_number] - week_offset
+    week_id = week_data[:week_number] + week_offset
 
     response = @conn.put("measurables/#{measurable_id}/week/#{week_id}", {value: score}.to_json)
     response.success?

data/lib/bloomy/operations/todos.rb

@@ -33,9 +33,7 @@ class Todo
   #   client.todo.list(meeting_id: 99)
   #   # => [{ id: 1, title: "New Todo", due_date: "2024-06-15", ... }]
   def list(user_id: nil, meeting_id: nil)
-    if user_id && meeting_id
-      raise ArgumentError, "Please provide either `user_id` or `meeting_id`, not both."
-    end
+    raise ArgumentError, "Please provide either `user_id` or `meeting_id`, not both." if user_id && meeting_id
 
     if meeting_id
       response = @conn.get("l10/#{meeting_id}/todos").body
@@ -76,7 +74,7 @@ class Todo
     {
       id: response["Id"],
       title: response["Name"],
-      meeting_name: response["Origin"],
+      meeting_title: response["Origin"],
       meeting_id: response["OriginId"],
       due_date: response["DueDate"],
       notes_url: response["DetailsUrl"]
@@ -92,7 +90,7 @@ class Todo
   #   #=> { status: 200 }
   def complete(todo_id)
     response = @conn.post("/api/v1/todo/#{todo_id}/complete?status=true")
-    {status: response.status}
+    response.success?
   end
 
   # Updates an existing todo
@@ -138,6 +136,7 @@ class Todo
     {
       id: todo["Id"],
       meeting_id: todo["OriginId"],
+      meeting_title: todo["Origin"],
       title: todo["Name"],
       notes_url: todo["DetailsUrl"],
       due_date: todo["DueDate"],

data/lib/bloomy/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Bloomy
-  VERSION = "0.4.2"
+  VERSION = "0.5.4"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: bloomy
 version: !ruby/object:Gem::Version
-  version: 0.4.2
+  version: 0.5.4
 platform: ruby
 authors:
 - Franccesco Orozco