luck_remover 0.1.0 → 0.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bc1b1530a599ee139347208b43f1c6fd6366c9ab7f78024702a5d8cc79293325
-  data.tar.gz: 2c99268c938c575639ff6fe9a06b098e7a419cfe17824d632d631fc462d7338e
+  metadata.gz: 1967a2ea5bdb16091fa57ae44a15e6a6d7ce41a3c23c91c1bad24a8520ff81b0
+  data.tar.gz: d8f41aff30a6b3b74ba81433b09e78bd46d41a6dc34688fb8ffc1917e60a0796
 SHA512:
-  metadata.gz: 9686dd5a3997e32c8045db01e564146f8d67cc5d612bef475d69a505c7a3a5eece606f52bfb0a90b80a282453158c2d26dd6fb3a657bf849c69ba37f43e1b9bf
-  data.tar.gz: 0c01f0d6f8dcbda67ccb8fb1096b7bada0713bc202792771e5c3edd27abaa71f027df33c3e9ca6877446e369a3fe8e4f60876053d06b4ea2936f2b09c48b86fc
+  metadata.gz: c75087a236f27d0007a6756c72c2bbedf53f31c8e06582f676756ea739876f08899bdf7889fe060b56568b39491edd3acfa85c177ed1bd81e1d4f2539e642b74
+  data.tar.gz: 766896c86c8f421efaf80e5e0986d409e381313ba5df97d9acf9a4f6d450d19123791396fd27eb9f1e1492742df1057be3858c54e59b647ff9f3050fdedce4d8

data/app/controllers/luck_remover/application_controller.rb

@@ -1,4 +1,5 @@
 module LuckRemover
   class ApplicationController < ActionController::Base
+    helper KeystoneUi::Engine.helpers
   end
 end

data/app/controllers/luck_remover/goals_controller.rb

@@ -0,0 +1,48 @@
+module LuckRemover
+  class GoalsController < LuckRemover.parent_controller.constantize
+    def index
+      @goals = goal_scope.order(:name)
+    end
+
+    def new
+      @goal = goal_scope.new
+      @metrics = LuckRemover.metrics.all
+    end
+
+    def create
+      @goal = goal_scope.new(goal_params)
+      return redirect_to goals_path if @goal.save
+
+      @metrics = LuckRemover.metrics.all
+      render :new, status: :unprocessable_content
+    end
+
+    def edit
+      @goal = goal_scope.find(params[:id])
+      @metrics = LuckRemover.metrics.all
+    end
+
+    def update
+      @goal = goal_scope.find(params[:id])
+      return redirect_to goals_path if @goal.update(goal_params)
+
+      @metrics = LuckRemover.metrics.all
+      render :edit, status: :unprocessable_content
+    end
+
+    def destroy
+      goal_scope.find(params[:id]).destroy!
+      redirect_to goals_path
+    end
+
+    private
+
+    def goal_params
+      params.expect(goal: [ :name, :metric_key, :target, :direction, :period ])
+    end
+
+    def goal_scope
+      Goal.for_owner(send(LuckRemover.current_owner_method))
+    end
+  end
+end

data/app/models/luck_remover/goal.rb

@@ -9,6 +9,7 @@ module LuckRemover
     enum :direction, { at_least: "at_least", at_most: "at_most" }, validate: true
     enum :period, { week: "week", month: "month" }, validate: true
 
+    validates :name, :target, presence: true
     validate :metric_is_registered
 
     def standing(now: Time.current)

data/app/views/luck_remover/goals/_form.html.erb

@@ -0,0 +1,12 @@
+<%= ui_form_field(attribute: "goal[name]", label: "Name", required: true, value: goal.name, errors: goal.errors[:name]) %>
+
+<% goal.errors[:metric_key].each do |message| %>
+  <%= ui_alert(message: "Metric #{message}", type: :error) %>
+<% end %>
+<% metrics.each do |metric| %>
+  <%= ui_radio_card(name: "goal[metric_key]", value: metric.key, label: metric.title, hint: metric.definition, checked: goal.metric_key.to_s == metric.key.to_s) %>
+<% end %>
+
+<%= ui_form_field(attribute: "goal[target]", label: "Target", type: :number, step: "any", required: true, value: goal.target, errors: goal.errors[:target]) %>
+<%= ui_form_field(attribute: "goal[direction]", label: "Direction", type: :select, required: true, options: [["At least", "at_least"], ["At most", "at_most"]], value: goal.direction, errors: goal.errors[:direction]) %>
+<%= ui_form_field(attribute: "goal[period]", label: "Period", type: :select, required: true, options: [["Week", "week"], ["Month", "month"]], value: goal.period, errors: goal.errors[:period]) %>

data/app/views/luck_remover/goals/edit.html.erb

@@ -0,0 +1,10 @@
+<%= ui_form_page(title: "Edit", subtitle: @goal.name) %>
+<%= ui_page(max_width: :md) do %>
+  <%= ui_form(action: luck_remover.goal_path(@goal), method: :patch) do %>
+    <%= render "luck_remover/goals/form", goal: @goal, metrics: @metrics %>
+    <%= ui_button(label: "Save") %>
+  <% end %>
+  <%= ui_action_menu do %>
+    <%= ui_action_menu_item(label: "Delete", href: luck_remover.goal_path(@goal), method: :delete) %>
+  <% end %>
+<% end %>

data/app/views/luck_remover/goals/index.html.erb

@@ -0,0 +1,21 @@
+<%= ui_show_page(title: "Goals") %>
+<%= ui_page(max_width: :xl) do %>
+  <%= ui_page_header(title: "Goals", action_url: luck_remover.new_goal_path, action_label: "Add") do |header| %>
+    <% header.action do %>
+      <%= ui_button(label: "Add", href: luck_remover.new_goal_path) %>
+    <% end %>
+  <% end %>
+  <% rows = @goals.map do |goal|
+       { name: goal.name,
+         edit_url: luck_remover.edit_goal_path(goal),
+         metric: LuckRemover.metric(goal.metric_key).title,
+         target: number_with_precision(goal.target, precision: 2, strip_insignificant_zeros: true),
+         direction: goal.direction.humanize,
+         period: goal.period.humanize }
+     end %>
+  <%= ui_data_table(
+        items: rows,
+        columns: [{ name: "Name" }, { metric: "Metric" }, { target: "Target" }, { direction: "Direction" }, { period: "Period" }],
+        empty_message: "No goals yet."
+      ) { |table| table.link(:name) { |row| row[:edit_url] } } %>
+<% end %>

data/app/views/luck_remover/goals/new.html.erb

@@ -0,0 +1,7 @@
+<%= ui_form_page(title: "Add goal") %>
+<%= ui_page(max_width: :md) do %>
+  <%= ui_form(action: luck_remover.goals_path) do %>
+    <%= render "luck_remover/goals/form", goal: @goal, metrics: @metrics %>
+    <%= ui_button(label: "Create") %>
+  <% end %>
+<% end %>

data/config/routes.rb

@@ -1,2 +1,3 @@
 LuckRemover::Engine.routes.draw do
+  resources :goals, except: [ :show ]
 end

data/lib/luck_remover/metric_registry.rb

@@ -19,6 +19,10 @@ module LuckRemover
       @stats.key?(key.to_sym)
     end
 
+    def all
+      @stats.values
+    end
+
     def fetch(key)
       @stats.fetch(key.to_sym)
     end

data/lib/luck_remover/version.rb

@@ -1,3 +1,3 @@
 module LuckRemover
-  VERSION = "0.1.0"
+  VERSION = "0.2.1"
 end

data/lib/luck_remover.rb

@@ -1,9 +1,13 @@
 require "readout"
+require "keystone_ui"
 require "luck_remover/version"
 require "luck_remover/engine"
 require "luck_remover/metric_registry"
 
 module LuckRemover
+  mattr_accessor :parent_controller, default: "LuckRemover::ApplicationController"
+  mattr_accessor :current_owner_method, default: nil
+
   def self.metrics
     @metrics ||= MetricRegistry.new
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: luck_remover
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.2.1
 platform: ruby
 authors:
 - Tyler Schneider
@@ -51,6 +51,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 0.2.0
+- !ruby/object:Gem::Dependency
+  name: keystone_ui
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.51.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.51.0
 description: LuckRemover stores goals that name a registered readout Stat, a target,
   a direction and a period, and reports whether the current period met each one.
 email:
@@ -64,6 +78,7 @@ files:
 - Rakefile
 - app/assets/stylesheets/luck_remover/application.css
 - app/controllers/luck_remover/application_controller.rb
+- app/controllers/luck_remover/goals_controller.rb
 - app/helpers/luck_remover/application_helper.rb
 - app/jobs/luck_remover/application_job.rb
 - app/mailers/luck_remover/application_mailer.rb
@@ -71,6 +86,10 @@ files:
 - app/models/luck_remover/goal.rb
 - app/models/luck_remover/standing.rb
 - app/views/layouts/luck_remover/application.html.erb
+- app/views/luck_remover/goals/_form.html.erb
+- app/views/luck_remover/goals/edit.html.erb
+- app/views/luck_remover/goals/index.html.erb
+- app/views/luck_remover/goals/new.html.erb
 - config/routes.rb
 - db/migrate/20261022212722_create_luck_remover_goals.rb
 - lib/luck_remover.rb