hr_lite 0.13.0 → 0.15.0

data/app/helpers/hr_lite/application_helper.rb

@@ -19,20 +19,28 @@ module HrLite
       { label: "Org",        path: :org_path,            match: [ "/org" ] },
       { label: "Kudos",      path: :kudos_path,          match: [ "/kudos" ] },
       { label: "Slips",      path: :salary_slips_path,   match: [ "/salary_slips" ] },
-      { label: "Career",     path: :career_path,         match: [ "/career", "/appraisals", "/profile" ] }
+      { label: "Career",     path: :career_path,         match: [ "/career", "/appraisals", "/profile" ] },
+      { label: "Documents",  path: :documents_path,      match: [ "/documents" ] },
+      { label: "Tax",        path: :tax_declaration_path, match: [ "/tax_declaration" ] },
+      { label: "Loans",      path: :loans_path,          match: [ "/loans" ] }
     ].freeze
 
     ADMIN_NAV_ITEMS = [
       { label: "Overview",        path: :admin_overview_path,        match: [ "/admin/overview" ] },
       { label: "Team attendance", path: :admin_attendances_path,     match: [ "/admin/attendances" ] },
       { label: "Approvals",       path: :admin_leave_requests_path,  match: [ "/admin/leave_requests", "/admin/leave_balances", "/admin/comp_off_requests", "/admin/regularization_requests" ] },
-      { label: "Reports",         path: :admin_reports_path,         match: [ "/admin/reports" ] }
+      { label: "Reports",         path: :admin_reports_path,         match: [ "/admin/reports" ] },
+      { label: "Claims",          path: :admin_expenses_path,        match: [ "/admin/expenses" ] },
+      { label: "Help desk",       path: :admin_hr_requests_path,     match: [ "/admin/hr_requests" ] },
+      { label: "Assets",          path: :admin_assets_path,          match: [ "/admin/assets" ] },
+      { label: "Joining & exits", path: :admin_checklists_path,      match: [ "/admin/checklists" ] }
     ].freeze
 
     LEADERSHIP_NAV_ITEMS = [
       { label: "Employees", path: :admin_employees_path,        match: [ "/admin/employees" ] },
       { label: "Settings",  path: :admin_leave_types_path,      match: [ "/admin/leave_types", "/admin/office_locations", "/admin/holidays", "/admin/setting" ] },
-      { label: "Audit",     path: :admin_audit_logs_path,       match: [ "/admin/audit_logs" ] }
+      { label: "Audit",     path: :admin_audit_logs_path,       match: [ "/admin/audit_logs" ] },
+      { label: "Documents", path: :admin_documents_path,        match: [ "/admin/documents" ] }
     ].freeze
 
     # Money screens, plus role management: granting somebody payroll is the
@@ -41,7 +49,9 @@ module HrLite
     SUPERADMIN_NAV_ITEMS = [
       { label: "Payroll", path: :admin_payroll_runs_path, match: [ "/admin/payroll_runs", "/admin/salary_slips" ] },
       { label: "Roles",   path: :admin_roles_path,        match: [ "/admin/roles" ] },
-      { label: "Rates",   path: :admin_statutory_rate_cards_path, match: [ "/admin/statutory_rate_cards" ] }
+      { label: "Rates",   path: :admin_statutory_rate_cards_path, match: [ "/admin/statutory_rate_cards" ] },
+      { label: "Tax",     path: :admin_tax_declarations_path, match: [ "/admin/tax_declarations" ] },
+      { label: "Loans",   path: :admin_loans_path,        match: [ "/admin/loans" ] }
     ].freeze
 
     # Only items whose routes exist yet (the nav grows with each phase).

data/app/models/hr_lite/approval_flow.rb

@@ -4,7 +4,7 @@ module HrLite
   class ApprovalFlow < ApplicationRecord
     include Audited
 
-    has_many :approval_steps, -> { order(:position) },
+    has_many :approval_steps, -> { order(:position) }, inverse_of: :flow,
              class_name: "HrLite::ApprovalStep", foreign_key: :flow_id, dependent: :destroy
     accepts_nested_attributes_for :approval_steps, allow_destroy: true
 

data/app/models/hr_lite/asset.rb

@@ -0,0 +1,57 @@
+module HrLite
+  # A laptop, a phone, a SIM. An asset nobody tracks is a cost nobody
+  # notices, and the moment it matters is somebody's last day.
+  class Asset < ApplicationRecord
+    include Audited
+
+    STATUSES = %w[available assigned returned lost damaged retired].freeze
+    CATEGORIES = %w[laptop phone sim id_card accessory vehicle other].freeze
+
+    has_many :asset_assignments, class_name: "HrLite::AssetAssignment", dependent: :destroy
+
+    validates :name, presence: true
+    validates :status, inclusion: { in: STATUSES }
+    validates :category, inclusion: { in: CATEGORIES }
+    validates :serial_number, uniqueness: { allow_blank: true }
+
+    scope :available, -> { where(status: "available") }
+    scope :out, -> { where(status: "assigned") }
+
+    STATUSES.each { |s| define_method("#{s}?") { status == s } }
+
+    def live_assignment = asset_assignments.find_by(returned_on: nil)
+
+    def holder = live_assignment&.user
+
+    def assign_to!(user, actor: nil, on: Date.current)
+      raise ActiveRecord::RecordInvalid.new(self), "already assigned" if live_assignment
+
+      transaction do
+        asset_assignments.create!(user_id: user.id, assigned_on: on, assigned_by_id: actor&.id)
+        update!(status: "assigned")
+      end
+      true
+    end
+
+    # `condition` records what came back — "screen cracked" is the difference
+    # between an asset returned and an asset written off.
+    def return!(actor: nil, on: Date.current, condition: nil, status: "available")
+      assignment = live_assignment
+      raise ActiveRecord::RecordInvalid.new(self), "not assigned" if assignment.nil?
+
+      transaction do
+        assignment.update!(returned_on: on, condition_note: condition.presence)
+        update!(status: status)
+      end
+      true
+    end
+
+    # Everything still out with somebody who has left — the report that
+    # should exist before an exit, not after.
+    def self.outstanding_for(user)
+      joins(:asset_assignments)
+        .where(hr_lite_asset_assignments: { user_id: user.id, returned_on: nil })
+        .distinct
+    end
+  end
+end

data/app/models/hr_lite/asset_assignment.rb

@@ -0,0 +1,24 @@
+module HrLite
+  # Who had it, when, and what state it came back in. Append-only: editing an
+  # assignment away is how a missing laptop stops being anybody's.
+  class AssetAssignment < ApplicationRecord
+    belongs_to :asset, class_name: "HrLite::Asset"
+    belongs_to :user, class_name: HrLite.config.user_class
+    belongs_to :assigned_by, class_name: HrLite.config.user_class, optional: true
+
+    validates :assigned_on, presence: true
+    validate :returned_after_it_was_given
+
+    scope :live, -> { where(returned_on: nil) }
+
+    def live? = returned_on.nil?
+
+    private
+
+    def returned_after_it_was_given
+      return if returned_on.nil? || assigned_on.nil? || returned_on >= assigned_on
+
+      errors.add(:returned_on, "cannot be before the day it was handed over")
+    end
+  end
+end

data/app/models/hr_lite/checklist_item.rb

@@ -0,0 +1,43 @@
+module HrLite
+  # One step for one person. Kept even when its template is later deleted —
+  # what a company DID on somebody's first day is history, and history should
+  # not change because a template was tidied up.
+  class ChecklistItem < ApplicationRecord
+    include Audited
+
+    belongs_to :template, class_name: "HrLite::ChecklistTemplate", optional: true
+    belongs_to :user, class_name: HrLite.config.user_class
+    belongs_to :completed_by, class_name: HrLite.config.user_class, optional: true
+
+    validates :kind, inclusion: { in: ChecklistTemplate::KINDS }
+    validates :title, presence: true
+
+    scope :outstanding, -> { where(completed_at: nil) }
+    scope :for_kind, ->(kind) { where(kind: kind.to_s) }
+    scope :overdue, ->(on = Date.current) { outstanding.where(due_on: ...on) }
+
+    def done? = completed_at.present?
+
+    def overdue?(on = Date.current) = !done? && due_on.present? && due_on < on
+
+    def complete!(actor:, note: nil)
+      update!(completed_at: Time.current, completed_by_id: actor&.id, note: note.presence)
+    end
+
+    def reopen!(actor:)
+      update!(completed_at: nil, completed_by_id: nil)
+    end
+
+    # Builds somebody's list from the active templates. Idempotent: running
+    # it twice does not double the list, so a re-run after adding a template
+    # adds only what is new.
+    def self.open_for!(user, kind:, anchor_date:)
+      ChecklistTemplate.for_kind(kind).filter_map do |template|
+        next if exists?(user_id: user.id, kind: kind.to_s, title: template.title)
+
+        create!(user_id: user.id, template: template, kind: kind.to_s, title: template.title,
+                due_on: anchor_date + template.due_offset_days)
+      end
+    end
+  end
+end

data/app/models/hr_lite/checklist_template.rb

@@ -0,0 +1,50 @@
+module HrLite
+  # "Collect the signed offer letter", "Revoke email access". The steps a
+  # company means to take every time and forgets once.
+  class ChecklistTemplate < ApplicationRecord
+    include Audited
+
+    KINDS = %w[onboarding offboarding].freeze
+
+    validates :kind, inclusion: { in: KINDS }
+    validates :title, presence: true
+    validate :owner_permission_is_declared
+
+    scope :active, -> { where(active: true) }
+    scope :for_kind, ->(kind) { active.where(kind: kind.to_s).order(:position, :id) }
+
+    def self.seed_defaults!
+      [
+        { kind: "onboarding", title: "Collect signed offer letter", position: 10,
+          owner_permission: "document.manage" },
+        { kind: "onboarding", title: "Collect PAN and Aadhaar", position: 20,
+          owner_permission: "document.manage" },
+        { kind: "onboarding", title: "Create payroll record and salary structure",
+          position: 30, owner_permission: "salary.manage", due_offset_days: 3 },
+        { kind: "onboarding", title: "Hand over laptop and access", position: 40,
+          owner_permission: "profile.manage" },
+        { kind: "offboarding", title: "Collect laptop and accessories", position: 10,
+          owner_permission: "profile.manage" },
+        { kind: "offboarding", title: "Revoke email and system access", position: 20,
+          owner_permission: "profile.manage" },
+        { kind: "offboarding", title: "Full and final settlement", position: 30,
+          owner_permission: "payroll.manage", due_offset_days: 30 },
+        { kind: "offboarding", title: "Issue relieving and experience letters",
+          position: 40, owner_permission: "document.manage", due_offset_days: 7 }
+      ].filter_map do |attributes|
+        next if exists?(kind: attributes[:kind], title: attributes[:title])
+
+        create!(attributes)
+        "#{attributes[:kind]}: #{attributes[:title]}"
+      end
+    end
+
+    private
+
+    def owner_permission_is_declared
+      return if owner_permission.blank? || Permissions.valid?(owner_permission)
+
+      errors.add(:owner_permission, "is not a known permission")
+    end
+  end
+end

data/app/models/hr_lite/employee_profile.rb

@@ -6,6 +6,7 @@ module HrLite
   class EmployeeProfile < ApplicationRecord
     include EncryptedMoney
     include Audited
+    after_create_commit :open_onboarding_checklist
 
     belongs_to :user, class_name: HrLite.config.user_class
     # Reporting line (L1). Optional; the org chart treats manager-less
@@ -152,5 +153,17 @@ module HrLite
 
       errors.add(:date_of_exit, "must be after joining")
     end
+
+    private
+
+    # Day one has a list, and it is the same list every time. Opened on
+    # create rather than by hand, because the one nobody remembers to start
+    # is the one that does not happen.
+    def open_onboarding_checklist
+      ChecklistItem.open_for!(user, kind: "onboarding", anchor_date: date_of_joining)
+    rescue => e
+      # A checklist must never be the reason an employee cannot be created.
+      Rails.logger.error("[hr_lite] onboarding checklist failed: #{e.class}: #{e.message}")
+    end
   end
 end

data/app/models/hr_lite/tax_declaration.rb

@@ -11,8 +11,11 @@ module HrLite
     STATUSES = %w[draft submitted verified rejected].freeze
     REGIMES = %w[new old].freeze
 
+    # inverse_of is load-bearing, not decoration: the foreign key is not
+    # derived from the class name, so without it a nested item on an UNSAVED
+    # declaration cannot see its parent and fails "declaration must exist".
     has_many :tax_declaration_items, -> { order(:section) },
-             class_name: "HrLite::TaxDeclarationItem",
+             class_name: "HrLite::TaxDeclarationItem", inverse_of: :declaration,
              foreign_key: :declaration_id, dependent: :destroy
     accepts_nested_attributes_for :tax_declaration_items, allow_destroy: true
 

data/app/views/hr_lite/admin/assets/index.html.erb

@@ -0,0 +1,59 @@
+<% content_for(:page_title) { "Assets" } %>
+<div class="hrl-page__head">
+  <h1 class="hrl-page__title">Assets</h1>
+  <div class="hrl-page__actions">
+    <a class="hrl-btn hrl-btn--primary" href="<%= hr_lite.new_admin_asset_path %>">Add asset</a>
+  </div>
+</div>
+
+<% if @outstanding.any? %>
+  <section class="hrl-card">
+    <h2 class="hrl-card__title">Currently out</h2>
+    <ul>
+      <% @outstanding.each do |assignment| %>
+        <li class="hrl-small">
+          <strong><%= assignment.asset.name %></strong> —
+          <%= hr_display_name(assignment.user) %>,
+          since <%= assignment.assigned_on.strftime("%d %b %Y") %>
+        </li>
+      <% end %>
+    </ul>
+  </section>
+<% end %>
+
+<section class="hrl-card">
+  <div class="hrl-table-wrap">
+    <table class="hrl-table hrl-table--stack">
+      <thead><tr><th>Asset</th><th>Status</th><th>With</th><th></th></tr></thead>
+      <tbody>
+        <% @assets.each do |asset| %>
+          <tr>
+            <td data-label="Asset">
+              <strong><%= asset.name %></strong>
+              <div class="hrl-small hrl-muted">
+                <%= asset.category.humanize %><%= " · #{asset.serial_number}" if asset.serial_number.present? %>
+              </div>
+            </td>
+            <td data-label="Status"><%= hrl_request_status_badge(asset.status) %></td>
+            <td data-label="With"><%= asset.holder ? hr_display_name(asset.holder) : "—" %></td>
+            <td data-label="">
+              <% if asset.assigned? %>
+                <%= form_with url: hr_lite.take_back_admin_asset_path(asset), method: :post do %>
+                  <%= text_field_tag :condition_note, nil, placeholder: "Condition" %>
+                  <%= submit_tag "Take back", class: "hrl-btn" %>
+                <% end %>
+              <% elsif asset.available? %>
+                <%= form_with url: hr_lite.assign_admin_asset_path(asset), method: :post do %>
+                  <%= select_tag :user_id,
+                        options_for_select(HrLite.employees.map { |u| [ hr_display_name(u), u.id ] }) %>
+                  <%= submit_tag "Hand over", class: "hrl-btn" %>
+                <% end %>
+              <% end %>
+            </td>
+          </tr>
+        <% end %>
+      </tbody>
+    </table>
+  </div>
+  <%= hrl_pagination %>
+</section>

data/app/views/hr_lite/admin/assets/new.html.erb

@@ -0,0 +1,23 @@
+<% content_for(:page_title) { "Add asset" } %>
+<div class="hrl-page__head"><h1 class="hrl-page__title">Add asset</h1></div>
+
+<section class="hrl-card">
+  <%= form_with model: @asset, url: hr_lite.admin_assets_path, method: :post, local: true do |f| %>
+    <%= render "hr_lite/shared/form_errors", record: @asset %>
+    <div class="hrl-field"><%= f.label :name %><%= f.text_field :name %></div>
+    <div class="hrl-field">
+      <%= f.label :category %>
+      <%= f.select :category, HrLite::Asset::CATEGORIES.map { |c| [ c.humanize, c ] } %>
+    </div>
+    <div class="hrl-field">
+      <%= f.label :serial_number, "Serial number (optional)" %>
+      <%= f.text_field :serial_number %>
+    </div>
+    <div class="hrl-field"><%= f.label :purchased_on %><%= f.date_field :purchased_on %></div>
+    <div class="hrl-field"><%= f.label :notes %><%= f.text_field :notes %></div>
+    <div class="hrl-form-actions">
+      <%= f.submit "Save", class: "hrl-btn hrl-btn--primary" %>
+      <a class="hrl-btn" href="<%= hr_lite.admin_assets_path %>">Cancel</a>
+    </div>
+  <% end %>
+</section>

data/app/views/hr_lite/admin/checklists/index.html.erb

@@ -0,0 +1,50 @@
+<% content_for(:page_title) { "Joining & exits" } %>
+<div class="hrl-page__head"><h1 class="hrl-page__title">Joining &amp; exits</h1></div>
+
+<section class="hrl-card">
+  <div class="hrl-row">
+    <% HrLite::ChecklistTemplate::KINDS.each do |kind| %>
+      <a class="hrl-btn <%= "hrl-btn--primary" if kind == @kind %>"
+         href="<%= hr_lite.admin_checklists_path(kind: kind) %>"><%= kind.humanize %></a>
+    <% end %>
+  </div>
+</section>
+
+<% if @overdue.any? %>
+  <section class="hrl-card">
+    <h2 class="hrl-card__title"><%= @overdue.size %> overdue</h2>
+    <p class="hrl-small hrl-muted">
+      An unreturned laptop and un-revoked access both outlive the employment.
+    </p>
+  </section>
+<% end %>
+
+<section class="hrl-card">
+  <% if @items.empty? %>
+    <p class="hrl-muted">Nothing outstanding.</p>
+  <% else %>
+    <div class="hrl-table-wrap">
+      <table class="hrl-table hrl-table--stack">
+        <thead><tr><th>Step</th><th>Who</th><th>Due</th><th></th></tr></thead>
+        <tbody>
+          <% @items.each do |item| %>
+            <tr>
+              <td data-label="Step">
+                <%= item.title %>
+                <% if item.overdue? %><span class="hrl-badge hrl-badge--bad">Overdue</span><% end %>
+              </td>
+              <td data-label="Who"><%= hr_display_name(item.user) %></td>
+              <td data-label="Due"><%= item.due_on&.strftime("%d %b %Y") || "—" %></td>
+              <td data-label="">
+                <%= form_with url: hr_lite.complete_admin_checklist_path(item), method: :post do %>
+                  <%= text_field_tag :note, nil, placeholder: "Note" %>
+                  <%= submit_tag "Done", class: "hrl-btn hrl-btn--primary" %>
+                <% end %>
+              </td>
+            </tr>
+          <% end %>
+        </tbody>
+      </table>
+    </div>
+  <% end %>
+</section>

data/app/views/hr_lite/admin/documents/index.html.erb

@@ -0,0 +1,89 @@
+<% content_for(:page_title) { "Documents" } %>
+<div class="hrl-page__head"><h1 class="hrl-page__title">Documents</h1></div>
+
+<section class="hrl-card">
+  <p class="hrl-muted hrl-small">
+    Seeing that a document exists and opening it are different things — a
+    passport is visible here, but the file itself only opens for whoever its
+    visibility allows.
+  </p>
+  <div class="hrl-row">
+    <a class="hrl-btn <%= "hrl-btn--primary" if @category.nil? %>"
+       href="<%= hr_lite.admin_documents_path %>">All</a>
+    <% @categories.each do |category| %>
+      <a class="hrl-btn <%= "hrl-btn--primary" if @category == category %>"
+         href="<%= hr_lite.admin_documents_path(category: category) %>"><%= category.humanize %></a>
+    <% end %>
+  </div>
+</section>
+
+<section class="hrl-card">
+  <h2 class="hrl-card__title">Waiting to be checked (<%= @pending.size %>)</h2>
+  <% if @pending.empty? %>
+    <p class="hrl-muted">Nothing pending.</p>
+  <% else %>
+    <div class="hrl-table-wrap">
+      <table class="hrl-table hrl-table--stack">
+        <thead><tr><th>Person</th><th>Document</th><th>File</th><th></th></tr></thead>
+        <tbody>
+          <% @pending.each do |doc| %>
+            <tr>
+              <td data-label="Person"><%= hr_display_name(doc.user) %></td>
+              <td data-label="Document">
+                <%= doc.title %>
+                <div class="hrl-small hrl-muted"><%= doc.category.humanize %></div>
+              </td>
+              <td data-label="File">
+                <% if doc.readable_by?(hr_current_user) && doc.file.attached? %>
+                  <a class="hrl-small" href="<%= hr_lite.download_document_path(doc) %>">Open</a>
+                <% else %>
+                  <span class="hrl-small hrl-muted">Not yours to open</span>
+                <% end %>
+              </td>
+              <td data-label="">
+                <%= button_to "Verify", hr_lite.verify_admin_document_path(doc),
+                      method: :post, class: "hrl-btn hrl-btn--primary" %>
+                <%= form_with url: hr_lite.reject_admin_document_path(doc), method: :post, local: true do %>
+                  <%= text_field_tag :note, nil, placeholder: "What is wrong with it" %>
+                  <%= submit_tag "Reject", class: "hrl-btn" %>
+                <% end %>
+              </td>
+            </tr>
+          <% end %>
+        </tbody>
+      </table>
+    </div>
+  <% end %>
+</section>
+
+<section class="hrl-card">
+  <h2 class="hrl-card__title">Everything on file</h2>
+  <div class="hrl-table-wrap">
+    <table class="hrl-table hrl-table--stack">
+      <thead><tr><th>Person</th><th>Document</th><th>Expires</th><th>Checked</th></tr></thead>
+      <tbody>
+        <% @documents.each do |doc| %>
+          <tr>
+            <td data-label="Person">
+              <a href="<%= hr_lite.admin_employee_documents_path(user_id: doc.user_id) %>">
+                <%= hr_display_name(doc.user) %>
+              </a>
+            </td>
+            <td data-label="Document"><%= doc.title %></td>
+            <td data-label="Expires">
+              <% if doc.expires_on %>
+                <span class="<%= "hrl-badge hrl-badge--bad" if doc.expired? %>">
+                  <%= doc.expires_on.strftime("%d %b %Y") %>
+                </span>
+              <% else %>—<% end %>
+            </td>
+            <td data-label="Checked">
+              <%= hrl_request_status_badge(doc.verification == "verified" ? "approved" : doc.verification) %>
+            </td>
+          </tr>
+        <% end %>
+      </tbody>
+    </table>
+  </div>
+  <%= hrl_pagination %>
+</section>

data/app/views/hr_lite/admin/documents/show.html.erb

@@ -0,0 +1,42 @@
+<% content_for(:page_title) { hr_display_name(@employee) } %>
+<div class="hrl-page__head">
+  <h1 class="hrl-page__title"><%= hr_display_name(@employee) %> — documents</h1>
+</div>
+
+<% if @missing.any? %>
+  <section class="hrl-card hrl-card--warn">
+    <h2 class="hrl-card__title">Not on file</h2>
+    <p><%= @missing.map(&:humanize).to_sentence %></p>
+  </section>
+<% end %>
+
+<section class="hrl-card">
+  <% if @documents.empty? %>
+    <p class="hrl-muted">Nothing uploaded.</p>
+  <% else %>
+    <div class="hrl-table-wrap">
+      <table class="hrl-table hrl-table--stack">
+        <thead><tr><th>Document</th><th>Expires</th><th>Checked</th><th></th></tr></thead>
+        <tbody>
+          <% @documents.each do |doc| %>
+            <tr>
+              <td data-label="Document">
+                <strong><%= doc.title %></strong>
+                <div class="hrl-small hrl-muted"><%= doc.category.humanize %></div>
+              </td>
+              <td data-label="Expires"><%= doc.expires_on&.strftime("%d %b %Y") || "—" %></td>
+              <td data-label="Checked">
+                <%= hrl_request_status_badge(doc.verification == "verified" ? "approved" : doc.verification) %>
+              </td>
+              <td data-label="">
+                <% if doc.readable_by?(hr_current_user) && doc.file.attached? %>
+                  <a class="hrl-small" href="<%= hr_lite.download_document_path(doc) %>">Open</a>
+                <% end %>
+              </td>
+            </tr>
+          <% end %>
+        </tbody>
+      </table>
+    </div>
+  <% end %>
+</section>

data/app/views/hr_lite/admin/expenses/index.html.erb

@@ -0,0 +1,54 @@
+<% content_for(:page_title) { "Claims" } %>
+<div class="hrl-page__head"><h1 class="hrl-page__title">Expense claims</h1></div>
+
+<section class="hrl-card">
+  <div class="hrl-row">
+    <% HrLite::Expense::STATUSES.each do |status| %>
+      <a class="hrl-btn <%= "hrl-btn--primary" if status == @status %>"
+         href="<%= hr_lite.admin_expenses_path(status: status) %>"><%= status.humanize %></a>
+    <% end %>
+  </div>
+</section>
+
+<section class="hrl-card">
+  <% if @expenses.empty? %>
+    <p class="hrl-muted">Nothing <%= @status %>.</p>
+  <% else %>
+    <div class="hrl-table-wrap">
+      <table class="hrl-table hrl-table--stack">
+        <thead><tr><th>Who</th><th>What</th><th>Amount</th><th></th></tr></thead>
+        <tbody>
+          <% @expenses.each do |expense| %>
+            <tr>
+              <td data-label="Who"><%= hr_display_name(expense.user) %></td>
+              <td data-label="What">
+                <%= expense.category.name %>
+                <div class="hrl-small hrl-muted">
+                  <%= expense.description %> · <%= expense.spent_on.strftime("%d %b %Y") %>
+                </div>
+              </td>
+              <td data-label="Amount" class="hrl-num"><%= hrl_money(expense.amount) %></td>
+              <td data-label="">
+                <% if expense.submitted? %>
+                  <%= form_with url: hr_lite.approve_admin_expense_path(expense), method: :post do %>
+                    <%= submit_tag "Approve", class: "hrl-btn hrl-btn--primary" %>
+                  <% end %>
+                  <%= form_with url: hr_lite.reject_admin_expense_path(expense), method: :post do %>
+                    <%= text_field_tag :decision_note, nil, placeholder: "Why not" %>
+                    <%= submit_tag "Reject", class: "hrl-btn" %>
+                  <% end %>
+                <% elsif expense.approved? %>
+                  <%= form_with url: hr_lite.reimburse_admin_expense_path(expense), method: :post do %>
+                    <%= text_field_tag :period_month, Date.current.strftime("%Y-%m"), size: 8 %>
+                    <%= submit_tag "Mark reimbursed", class: "hrl-btn" %>
+                  <% end %>
+                <% end %>
+              </td>
+            </tr>
+          <% end %>
+        </tbody>
+      </table>
+    </div>
+    <%= hrl_pagination %>
+  <% end %>
+</section>

data/app/views/hr_lite/admin/hr_requests/index.html.erb

@@ -0,0 +1,41 @@
+<% content_for(:page_title) { "Help desk" } %>
+<div class="hrl-page__head"><h1 class="hrl-page__title">Help desk</h1></div>
+
+<section class="hrl-card">
+  <div class="hrl-row">
+    <% HrLite::HrRequest::STATUSES.each do |status| %>
+      <a class="hrl-btn <%= "hrl-btn--primary" if status == @status %>"
+         href="<%= hr_lite.admin_hr_requests_path(status: status) %>"><%= status.humanize %></a>
+    <% end %>
+  </div>
+</section>
+
+<section class="hrl-card">
+  <% if @requests.empty? %>
+    <p class="hrl-muted">Nothing <%= @status.humanize.downcase %>.</p>
+  <% else %>
+    <div class="hrl-table-wrap">
+      <table class="hrl-table hrl-table--stack">
+        <thead><tr><th>Request</th><th>Who</th><th>With</th><th></th></tr></thead>
+        <tbody>
+          <% @requests.each do |request| %>
+            <tr>
+              <td data-label="Request">
+                <a href="<%= hr_lite.admin_hr_request_path(request) %>"><%= request.subject %></a>
+                <div class="hrl-small hrl-muted"><%= request.category_label %></div>
+              </td>
+              <td data-label="Who"><%= hr_display_name(request.user) %></td>
+              <td data-label="With">
+                <%= request.assigned_to ? hr_display_name(request.assigned_to) : "—" %>
+              </td>
+              <td data-label="">
+                <a class="hrl-small" href="<%= hr_lite.admin_hr_request_path(request) %>">Open</a>
+              </td>
+            </tr>
+          <% end %>
+        </tbody>
+      </table>
+    </div>
+    <%= hrl_pagination %>
+  <% end %>
+</section>