Refactor admin users page, add quick stats

app/controllers/admin/ldap_users_controller.rb

@@ -2,43 +2,18 @@ class Admin::LdapUsersController < Admin::BaseController
   before_action :set_current_section
 
   def index
-    attributes = %w{dn cn uid mail admin}
+    ldap = LdapService.new
-    filter = Net::LDAP::Filter.eq("uid", "*")
-
     @ou = params[:ou] || "kosmos.org"
-    treebase = "ou=#{@ou},cn=users,dc=kosmos,dc=org"
+    @orgs = ldap.fetch_organizations
-
+    @entries = ldap.fetch_users(ou: @ou)
-    entries = ldap_client.search(base: treebase, filter: filter, attributes: attributes)
+    @stats = {
-    entries.sort_by! { |e| e.cn[0] }
+      users_confirmed: User.where(ou: @ou).confirmed.count,
-
+      users_pending: User.where(ou: @ou).pending.count
-    @entries = entries.collect do |e|
-      {
-        uid: e.uid.first,
-        mail: e.try(:mail) ? e.mail.first : nil,
-        admin: e.try(:admin) ? 'admin' : nil
-        # password: e.userpassword.first
     }
   end
-    # ldap_client.get_operation_result
-  end
 
   private
 
-  def ldap_client
-    ldap_client ||= Net::LDAP.new host: ldap_config['host'],
-      port: ldap_config['port'],
-      # encryption: ldap_config['ssl'],
-      auth: {
-        method: :simple,
-        username: ldap_config['admin_user'],
-        password: ldap_config['admin_password']
-      }
-  end
-
-  def ldap_config
-    ldap_config ||= YAML.load(ERB.new(File.read("#{Rails.root}/config/ldap.yml")).result)[Rails.env]
-  end
-
   def set_current_section
     @current_section = :ldap_users
   end

app/models/user.rb

@@ -15,6 +15,9 @@ class User < ApplicationRecord
   validates_uniqueness_of :email
   validates :email, email: true
 
+  scope :confirmed, -> { where.not(confirmed_at: nil) }
+  scope :pending,   -> { where(confirmed_at: nil) }
+
   lockbox_encrypts :ln_login
   lockbox_encrypts :ln_password
 

app/views/admin/ldap_users/index.html.erb

@@ -1,16 +1,35 @@
 <%= render HeaderComponent.new(title: "LDAP Users: #{@ou}") %>
 
 <%= render MainSimpleComponent.new do %>
-  <h3 class="hidden">Domains</h3>
+  <section>
-  <ul class="mb-10">
+    <%= render QuickstatsContainerComponent.new do %>
-    <li class="inline-block">
+      <%= render QuickstatsItemComponent.new(
-      <%= link_to 'kosmos.org', admin_ldap_users_path, class: "ks-text-link" %>
+          type: :number,
-    </li>
+          title: 'Confirmed',
-    <li class="inline-block ml-6">
+          value: @stats[:users_confirmed],
-      <%= link_to '5apps.com', admin_ldap_users_path(ou: '5apps.com'), class: "ks-text-link" %>
+      ) %>
-    </li>
+      <%= render QuickstatsItemComponent.new(
-  </ul>
+          type: :number,
+          title: 'Pending',
+          value: @stats[:users_pending],
+      ) %>
+    <% end %>
+  </section>
 
+  <% if @orgs.length > 1 %>
+    <section>
+      <h3 class="hidden">Domains</h3>
+      <ul>
+        <% @orgs.each do |org| %>
+          <li class="inline-block">
+            <%= link_to org[:ou], admin_ldap_users_path(ou: org[:ou]), class: "ks-text-link" %>
+          </li>
+        <% end %>
+      </ul>
+    </section>
+  <% end %>
+
+  <section>
     <table>
       <thead>
         <tr>
@@ -31,4 +50,5 @@
         <% end %>
       </tbody>
     </table>
+  </section>
 <% end %>

spec/components/wallet_summary_component_spec.rb

@@ -5,7 +5,7 @@ RSpec.describe WalletSummaryComponent, type: :component do
     expect(
       render_inline(described_class.new(balance: 2301000)) {}.css("section").to_html
     ).to include(
-      "2,301,000 sats"
+      "2,301,000"
     )
   end
 end