From 1cad60b77b31a6232de06f774a49a4c4a75ea111 Mon Sep 17 00:00:00 2001
From: mayra lucia navarro <luciagirasoles@gmail.com>
Date: Tue, 15 Aug 2023 02:23:43 -0500
Subject: [PATCH 1/4] Create Factory for events

---
 .../admin/events_controller_spec.rb           | 39 +++++++------
 .../controllers/api/events_controller_spec.rb | 55 ++++++-------------
 spec/system/managing_events_spec.rb           |  2 +-
 3 files changed, 38 insertions(+), 58 deletions(-)

diff --git a/spec/controllers/admin/events_controller_spec.rb b/spec/controllers/admin/events_controller_spec.rb
index 177bbea2..ec847a7b 100644
--- a/spec/controllers/admin/events_controller_spec.rb
+++ b/spec/controllers/admin/events_controller_spec.rb
@@ -3,7 +3,7 @@
 require './spec/rails_helper'
 
 RSpec.describe Admin::EventsController, type: :controller do
-  describe '#index' do
+  describe 'GET #index' do
     context 'when user is not authenticated' do
       it 'returns 302' do
         get :index
@@ -33,7 +33,7 @@
     end
   end
 
-  describe '#edit' do
+  describe 'GET #edit' do
     context 'when user is not authenticated' do
       let(:event) { FactoryBot.create(:event) }
 
@@ -50,7 +50,7 @@
       let(:event) { FactoryBot.create(:event) }
 
       it 'returns 401' do
-        get :edit, params: {id: event.id}
+        get :edit, params: { id: event.id }
         expect(response).to have_http_status(401)
       end
     end
@@ -61,18 +61,18 @@
       let(:event) { FactoryBot.create(:event) }
 
       it 'returns 200' do
-        get :edit, params: {id: event.id}
+        get :edit, params: { id: event.id }
         expect(response).to have_http_status(200)
       end
     end
   end
 
-  describe '#update' do
+  describe 'PUT #update' do
     context 'when user is not authenticated' do
       let(:event) { FactoryBot.create(:event) }
 
       it 'returns 302' do
-        post :update, params: { id: event.id }
+        put :update, params: { id: event.id }
         expect(response).to have_http_status(302)
         expect(response).to redirect_to(new_user_session_path)
       end
@@ -109,7 +109,7 @@
     end
   end
 
-  describe '#create' do
+  describe 'POST #create' do
     context 'when user is not authenticated' do
       it 'returns 302' do
         post :create
@@ -134,7 +134,14 @@
 
       it 'returns 302' do
         post :create,
-        params: { event: { title: 'My event', description: 'Great event', date: 1.month.from_now, type: 'Meetup' } }
+             params: {
+               event: {
+                 title: 'My event',
+                 description: 'Great event',
+                 date: 1.month.from_now,
+                 type: 'Meetup',
+               },
+             }
 
         expect(response).to redirect_to(admin_events_path)
       end
@@ -146,18 +153,16 @@
       let(:user) { create(:user, :admin) }
       let(:event) { create(:event) }
 
-      before(:each) do
-        sign_in user
-      end
+      before(:each) { sign_in user }
 
       it 'redirects to index page when event exists' do
-        delete :destroy, params: {id: event.id}
+        delete :destroy, params: { id: event.id }
         expect(response).to redirect_to(admin_events_path)
         expect(Event.exists?(event.id)).to be(false)
       end
 
       it 'returns 404 when the event does not exist' do
-        delete :destroy, params: {id: 'fakefake'}
+        delete :destroy, params: { id: 'fakefake' }
         expect(response).to have_http_status(404)
       end
     end
@@ -166,12 +171,10 @@
       let(:user) { create(:user) }
       let(:event) { create(:event) }
 
-      before(:each) do
-        sign_in user
-      end
+      before(:each) { sign_in user }
 
       it 'returns 401' do
-        delete :destroy, params: {id: event.id}
+        delete :destroy, params: { id: event.id }
         expect(response).to have_http_status(401)
         expect(Event.exists?(event.id)).to be(true)
       end
@@ -181,7 +184,7 @@
       let(:event) { create(:event) }
 
       it 'redirects to login' do
-        delete :destroy, params: {id: event.id}
+        delete :destroy, params: { id: event.id }
         expect(response).to redirect_to(new_user_session_path)
         expect(Event.exists?(event.id)).to be(true)
       end
diff --git a/spec/controllers/api/events_controller_spec.rb b/spec/controllers/api/events_controller_spec.rb
index fcf0ad2a..21fe40b9 100644
--- a/spec/controllers/api/events_controller_spec.rb
+++ b/spec/controllers/api/events_controller_spec.rb
@@ -13,30 +13,19 @@
 
   describe 'GET #past' do
     before do
-      july_meetup =
-        Meetup.create(
-          title: 'July Meetup',
-          location: 'virtual',
-          date: DateTime.new(2021, 7, 31, 16).utc,
-        )
-      Meetup.create(
-        title: 'August Meetup',
-        location: 'virtual',
-        date: DateTime.new(2021, 8, 31, 16).utc,
-      )
+      july_meetup = create(:event, title: 'July Meetup', date: DateTime.new(2021, 7, 31, 16).utc)
+      create(:event, title: 'August Meetup', date: DateTime.new(2021, 8, 31, 16).utc)
       Panel.create(title: 'Future Panel', location: 'Denver, Colorado', date: DateTime.now + 1.week)
 
       speaker =
-        Speaker.create(
+        create(
+          :speaker,
           name: 'Speaker Name',
-          tagline: 'Software Developer',
-          bio: 'Lorem Ipsum',
-          image_url: 'https://picsum.photos/200',
           links: {
             twitter: 'http://example.com/twitter-link',
             mastodon: 'http://example.com/mastodon-link',
-            personal_website: 'http://example.com/personal-website-link',
-          }
+            website: 'http://example.com/personal-website-link',
+          },
         )
       EventSpeaker.create(
         event: july_meetup,
@@ -71,11 +60,13 @@
       july_meetup = body['data']['2021']['July'].first
 
       expect(july_meetup['speakers'].first['name']).to eq('Speaker Name')
-      expect(july_meetup['speakers'].first['links']).to eq({
-        'twitter' => 'http://example.com/twitter-link',
-        'mastodon' => 'http://example.com/mastodon-link',
-        'personal_website' => 'http://example.com/personal-website-link',
-      })
+      expect(july_meetup['speakers'].first['links']).to eq(
+        {
+          'twitter' => 'http://example.com/twitter-link',
+          'mastodon' => 'http://example.com/mastodon-link',
+          'website' => 'http://example.com/personal-website-link',
+        },
+      )
     end
 
     it 'includes talk titles' do
@@ -108,25 +99,11 @@
   describe 'GET #upcoming' do
     before do
       august_meetup =
-        Meetup.create(
-          title: 'August Meetup',
-          location: 'virtual',
-          date: DateTime.new(2022, 8, 31, 16).utc,
-        )
-      Meetup.create(
-        title: 'September Meetup',
-        location: 'virtual',
-        date: DateTime.new(2022, 9, 25, 16).utc,
-      )
+        create(:event, title: 'August Meetup', date: DateTime.new(2022, 8, 31, 16).utc)
+      create(:event, title: 'September Meetup', date: DateTime.new(2022, 9, 25, 16).utc)
       Panel.create(title: 'Future Panel', location: 'Denver, Colorado', date: DateTime.now - 1.week)
 
-      speaker =
-        Speaker.create(
-          name: 'Speaker Name',
-          tagline: 'Software Developer',
-          bio: 'Lorem Ipsum',
-          image_url: 'https://picsum.photos/200',
-        )
+      speaker = create(:speaker, name: 'Speaker Name')
       EventSpeaker.create(
         event: august_meetup,
         speaker: speaker,
diff --git a/spec/system/managing_events_spec.rb b/spec/system/managing_events_spec.rb
index 91cdfecc..841439d1 100644
--- a/spec/system/managing_events_spec.rb
+++ b/spec/system/managing_events_spec.rb
@@ -36,7 +36,7 @@
     describe 'with invalid data' do
       it 'shows errors' do
         click_on 'Save'
-        # byebug
+
         expect(page).to have_current_path(admin_events_path)
         expect(page).to have_text("Title can't be blank")
       end

From 216fafb12c66221d97df4e294935685c3ede02de Mon Sep 17 00:00:00 2001
From: mayra lucia navarro <luciagirasoles@gmail.com>
Date: Tue, 15 Aug 2023 02:25:32 -0500
Subject: [PATCH 2/4] Generate Speaker module for admin - Add HTTParty to
 validate links - Add views, controllers, policies, model for speaker

---
 Gemfile                                       |   3 +
 Gemfile.lock                                  |  11 +-
 app/controllers/admin/speakers_controller.rb  |  95 +++++++++++
 app/helpers/admin/speakers_helper.rb          |   6 +
 app/models/speaker.rb                         |  38 +++++
 app/policies/speaker_policy.rb                |  26 +++
 app/views/admin/events/_form.html.erb         |   2 +-
 app/views/admin/events/edit.html.erb          |   2 +-
 app/views/admin/events/index.html.erb         |  63 +++----
 app/views/admin/events/new.html.erb           |   2 +-
 app/views/admin/speakers/_form.html.erb       |  46 +++++
 app/views/admin/speakers/edit.html.erb        |   6 +
 app/views/admin/speakers/index.html.erb       |  51 ++++++
 .../speakers/index/_speaker_links.html.erb    |  25 +++
 app/views/admin/speakers/new.html.erb         |   7 +
 app/views/admin/speakers/show.html.erb        |  10 ++
 app/views/layouts/admin/_flashes.html.erb     |   2 +-
 app/views/layouts/admin/_form_errors.html.erb |   8 +-
 app/views/layouts/admin/_header.html.erb      |  11 +-
 config/routes.rb                              |   2 +
 .../admin/speakers_controller_spec.rb         | 160 ++++++++++++++++++
 spec/factories/speakers.rb                    |  24 +++
 spec/models/speaker_spec.rb                   |  51 +++++-
 spec/rails_helper.rb                          |   6 +
 spec/system/managing_speakers_spec.rb         |  45 +++++
 25 files changed, 657 insertions(+), 45 deletions(-)
 create mode 100644 app/controllers/admin/speakers_controller.rb
 create mode 100644 app/helpers/admin/speakers_helper.rb
 create mode 100644 app/policies/speaker_policy.rb
 create mode 100644 app/views/admin/speakers/_form.html.erb
 create mode 100644 app/views/admin/speakers/edit.html.erb
 create mode 100644 app/views/admin/speakers/index.html.erb
 create mode 100644 app/views/admin/speakers/index/_speaker_links.html.erb
 create mode 100644 app/views/admin/speakers/new.html.erb
 create mode 100644 app/views/admin/speakers/show.html.erb
 create mode 100644 spec/controllers/admin/speakers_controller_spec.rb
 create mode 100644 spec/factories/speakers.rb
 create mode 100644 spec/system/managing_speakers_spec.rb

diff --git a/Gemfile b/Gemfile
index 4ea6e88f..46ef2662 100644
--- a/Gemfile
+++ b/Gemfile
@@ -34,6 +34,9 @@ gem 'google_drive', git: 'https://github.com/wnbrb/google-drive-ruby.git', branc
 # cuts off a string of HTML
 gem 'truncate_html', '~> 0.9.3'
 
+# quickly call web links
+gem 'httpparty', '~> 0.2'
+
 group :development, :test do
   gem 'byebug', platforms: %i[mri mingw x64_mingw]
   gem 'faker', '~> 2.18.0'
diff --git a/Gemfile.lock b/Gemfile.lock
index 27c873d3..72ae619e 100644
--- a/Gemfile.lock
+++ b/Gemfile.lock
@@ -120,7 +120,7 @@ GEM
       railties (>= 5.0.0)
     faker (2.18.0)
       i18n (>= 1.6, < 2)
-    faraday (2.7.10)
+    faraday (2.7.4)
       faraday-net_http (>= 2.0, < 3.1)
       ruby2_keywords (>= 0.0.4)
     faraday-mashify (0.1.1)
@@ -154,8 +154,13 @@ GEM
       os (>= 0.9, < 2.0)
       signet (>= 0.16, < 2.a)
     hashie (5.0.0)
+    httparty (0.21.0)
+      mini_mime (>= 1.0.0)
+      multi_xml (>= 0.5.2)
     httpclient (2.8.3)
-    i18n (1.14.1)
+    httpparty (0.2.0)
+      httparty (> 0)
+    i18n (1.12.0)
       concurrent-ruby (~> 1.0)
     io-console (0.6.0)
     irb (1.7.4)
@@ -188,6 +193,7 @@ GEM
     minitest (5.19.0)
     msgpack (1.7.2)
     multi_json (1.15.0)
+    multi_xml (0.6.0)
     multipart-post (2.3.0)
     net-imap (0.3.7)
       date
@@ -396,6 +402,7 @@ DEPENDENCIES
   factory_bot_rails (~> 6.2.0)
   faker (~> 2.18.0)
   google_drive!
+  httpparty (~> 0.2)
   jbuilder (~> 2.7)
   jsbundling-rails (~> 1.1)
   jwt (>= 2.6.0)
diff --git a/app/controllers/admin/speakers_controller.rb b/app/controllers/admin/speakers_controller.rb
new file mode 100644
index 00000000..abcb7e11
--- /dev/null
+++ b/app/controllers/admin/speakers_controller.rb
@@ -0,0 +1,95 @@
+# frozen_string_literal: true
+module Admin
+  class SpeakersController < AdminController
+    before_action :authorize_event
+    before_action :set_speaker, only: %w[show edit update destroy]
+
+    # GET /admin/speakers
+    def index
+      @speakers = Speaker.ordered_by_name
+    end
+
+    # GET /admin/speakers/1
+    def show; end
+
+    # GET /admin/speakers/new
+    def new
+      @speaker = Speaker.new
+    end
+
+    # GET /admin/speakers/1/edit
+    def edit
+      render_not_found unless @speaker
+    end
+
+    # POST /admin/speakers
+    def create
+      @speaker = Speaker.new(speaker_params)
+
+      respond_to do |format|
+        if @speaker.save
+          format.html do
+            redirect_to edit_admin_speaker_url(@speaker),
+                        notice: 'Speaker was successfully created.'
+          end
+        else
+          format.html { render :new, status: :unprocessable_entity }
+        end
+      end
+    end
+
+    # PATCH/PUT /admin/speakers/1
+    def update
+      respond_to do |format|
+        if @speaker.update(speaker_params)
+          format.html do
+            redirect_to edit_admin_speaker_url(@speaker),
+                        notice: 'Speaker was successfully updated.'
+          end
+          format.json { render :show, status: :ok, location: @speaker }
+        else
+          format.html { render :edit, status: :unprocessable_entity }
+          format.json { render json: @speaker.errors, status: :unprocessable_entity }
+        end
+      end
+    end
+
+    # DELETE /admin/speakers/1
+    # def destroy
+    #   @speaker.destroy
+
+    #   respond_to do |format|
+    #     format.html do
+    #       redirect_to admin_speakers_url, notice: 'Speaker was successfully destroyed.'
+    #     end
+    #   end
+    # end
+
+    private
+
+    def authorize_event
+      authorize Event
+    end
+
+    # Use callbacks to share common setup or constraints between actions.
+    def set_speaker
+      @speaker = Speaker.find(params[:id])
+    end
+
+    # Only allow a list of trusted parameters through.
+    def speaker_params
+      params.require(:speaker).permit(
+        :name,
+        :bio,
+        :tagline,
+        :image_url,
+        :github,
+        :linkedin,
+        :mastodon,
+        :website,
+        :twitter,
+        :other,
+      )
+    end
+  end
+end
diff --git a/app/helpers/admin/speakers_helper.rb b/app/helpers/admin/speakers_helper.rb
new file mode 100644
index 00000000..d430c448
--- /dev/null
+++ b/app/helpers/admin/speakers_helper.rb
@@ -0,0 +1,6 @@
+# frozen_string_literal: true
+module Admin
+  module SpeakersHelper
+    #Add helpers
+  end
+end
diff --git a/app/models/speaker.rb b/app/models/speaker.rb
index 71aed937..251be285 100644
--- a/app/models/speaker.rb
+++ b/app/models/speaker.rb
@@ -1,8 +1,46 @@
 # frozen_string_literal: true
 
 class Speaker < ApplicationRecord
+  include HTTParty
+  SOCIAL_MEDIA_LINKS = %w[github linkedin mastodon twitter website other].freeze
+
   has_many :event_speakers, dependent: :destroy
   has_many :events, through: :event_speakers
 
   validates :name, :bio, :image_url, presence: true
+  validate :links, :validate_social_media_brand
+  validate :url_exists?, if: ->(s) { s.links.compact_blank.present? }
+
+  store :links, accessors: SOCIAL_MEDIA_LINKS, coder: JSON
+
+  before_validation :format_links
+
+  scope :ordered_by_name, -> { order(:name) }
+
+  def validate_social_media_brand
+    return if links.blank? || links.keys.all? { |key| SOCIAL_MEDIA_LINKS.include?(key) }
+
+    errors.add(:links, 'This social media is not allowed')
+  end
+
+  def format_links
+    return if links.blank?
+
+    links.transform_keys!(&:downcase)
+  end
+
+  def url_exists?
+    links.each { |_, url| errors.add(:links, 'This url is not valid') unless url_valid?(url) }
+  end
+
+  def empty_links?
+    links.compact_blank.empty?
+  end
+
+  private
+
+  def url_valid?(url)
+    response = HTTParty.get(url)
+    response.code == 200
+  end
 end
diff --git a/app/policies/speaker_policy.rb b/app/policies/speaker_policy.rb
new file mode 100644
index 00000000..80fc3dc2
--- /dev/null
+++ b/app/policies/speaker_policy.rb
@@ -0,0 +1,26 @@
+# frozen_string_literal: true
+class SpeakerPolicy < ApplicationPolicy
+  def index?
+    user.admin?
+  end
+
+  def new?
+    user.admin?
+  end
+
+  def create?
+    user.admin?
+  end
+
+  def edit?
+    user.admin?
+  end
+
+  def show?
+    edit?
+  end
+
+  def update?
+    user.admin?
+  end
+end
diff --git a/app/views/admin/events/_form.html.erb b/app/views/admin/events/_form.html.erb
index 80cef8aa..74105c8e 100644
--- a/app/views/admin/events/_form.html.erb
+++ b/app/views/admin/events/_form.html.erb
@@ -1,4 +1,4 @@
-  <%= form_for [:admin, @event.becomes(Event)] do |f| %>
+  <%= form_for [:admin, event.becomes(Event)] do |f| %>
     <%= render 'layouts/admin/form_errors', object: f.object %>
     <div class="field form-floating mb-4">
       <%= f.text_field :title, class: "form-control" %>
diff --git a/app/views/admin/events/edit.html.erb b/app/views/admin/events/edit.html.erb
index 4208a30c..3b8fdfd0 100644
--- a/app/views/admin/events/edit.html.erb
+++ b/app/views/admin/events/edit.html.erb
@@ -1,6 +1,6 @@
 <div class="row">
   <div class="col col-lg-6">
       <h1>Edit event</h1>
-      <%=render 'form'%>
+      <%=render 'form', event: @event %>
   </div>
 </div>
diff --git a/app/views/admin/events/index.html.erb b/app/views/admin/events/index.html.erb
index 98930c17..c48313f6 100644
--- a/app/views/admin/events/index.html.erb
+++ b/app/views/admin/events/index.html.erb
@@ -1,7 +1,7 @@
 <h1 class="mb-4">Event List</h1>
 <div class="row">
   <div class="col">
-    <%= link_to new_admin_event_path, class: "btn btn-primary btn-lg mb-3" do %>
+    <%= link_to new_admin_event_path, class: "btn btn-primary btn-md mb-4" do %>
       Create new Event
       <i class="bi bi-plus fw-semibold"></i>
     <% end %>
@@ -9,34 +9,37 @@
 </div>
 <div class="row">
   <div class="col">
-    <table class="table text-center">
-      <thead class="border-2">
-      <tr>
-          <th scope="col" class="p-3">Title</th>
-          <th scope="col" class="p-3">Type</th>
-          <th scope="col" class="p-3">Speakers</th>
-          <th scope="col" class="p-3">Date</th>
-          <th scope="col" class="p-3">Location</th>
-          <th scope="col" class="p-3">Description</th>
-          <th scope="col" class="p-3">Actions</th>
-      </tr>
-      </thead>
-      <tbody>
-        <% @events.each do |event| %>
-          <tr class="border-2">
-            <td class="p-3"><%= event.title %></td>
-            <td class="p-3"><%= event.type %></td>
-            <td class="p-3"><%= event.speakers.map(&:name).join(", ") %></td>
-            <td class="p-3"><%= event.date.strftime('%B %d, %Y %I:%M%p %:z') %></td>
-            <td class="p-3"><%= event.location %></td>
-            <td class="p-3"><%= truncate_html(event.description) %></td>
-            <td class="p-3">
-              <%= link_to 'Edit', edit_admin_event_path(event) %>
-              <%= link_to 'Delete', admin_event_path(event), method: :delete %>
-            </td>
-          </tr>
-        <% end %>
-      </tbody>
-    </table>
+    <div class="table-responsive">
+      <table class="table table-sm text-center table-hover">
+        <thead class="border-2">
+        <tr class="table-primary">
+            <th scope="col" class="p-3">Title</th>
+            <th scope="col" class="p-3">Type</th>
+            <th scope="col" class="p-3">Speakers</th>
+            <th scope="col" class="p-3">Date</th>
+            <th scope="col" class="p-3">Location</th>
+            <th scope="col" class="p-3">Description</th>
+            <th scope="col" class="p-3">Actions</th>
+        </tr>
+        </thead>
+        <tbody>
+          <% @events.each do |event| %>
+
+            <tr class="border-2">
+              <td class="p-3"><%= event.title %></td>
+              <td class="p-3"><%= event.type %></td>
+              <td class="p-3"><%= event.speakers.map(&:name).join(", ") %></td>
+              <td class="p-3"><%= event.date.strftime('%B %d, %Y %I:%M%p %:z') %></td>
+              <td class="p-3"><%= event.location %></td>
+              <td class="p-3"><%= truncate_html(event.description) %></td>
+              <td class="p-3">
+                <%= link_to 'Edit', edit_admin_event_path(event) %>
+                <%= button_to "Delete", admin_event_url(event), form_class: "", class: "link-primary border-0 bg-transparent text-decoration-underline", method: :delete %>
+              </td>
+            </tr>
+          <% end %>
+        </tbody>
+      </table>
+    </div>
   </div>
 </div>
diff --git a/app/views/admin/events/new.html.erb b/app/views/admin/events/new.html.erb
index a6f5ed2d..4f4c1d0a 100644
--- a/app/views/admin/events/new.html.erb
+++ b/app/views/admin/events/new.html.erb
@@ -1,6 +1,6 @@
 <div class="row">
   <div class="col col-lg-6">
     <h1>New Event</h1>
-    <%= render 'form'%>
+    <%= render 'form', event: @event %>
   </div>
 </div>
diff --git a/app/views/admin/speakers/_form.html.erb b/app/views/admin/speakers/_form.html.erb
new file mode 100644
index 00000000..5592d23d
--- /dev/null
+++ b/app/views/admin/speakers/_form.html.erb
@@ -0,0 +1,46 @@
+<%= form_for [:admin, speaker] do |f| %>
+  <%= render "layouts/admin/form_errors", object: f.object %>
+  <div class="field form-floating mb-4">
+    <%= f.text_field :name, class: "form-control" %>
+    <%= f.label :name %>
+  </div>
+  <div class="field form-floating mb-4">
+    <%= f.text_area :bio, rows: 40, class: "form-control", style: "height: 200px" %>
+    <%= f.label :bio %>
+  </div>
+  <div class="field form-floating mb-4">
+    <%= f.text_field :tagline, class: "form-control" %>
+    <%= f.label :tagline %>
+  </div>
+  <div class="field form-floating mb-4">
+    <%= f.url_field :image_url, class: "form-control" %>
+    <%= f.label :image_url, "Image link" %>
+  </div>
+
+  <div class="accordion mb-4">
+    <div class="accordion-item">
+      <h2 class="accordion-header" id="headingLinks">
+        <% accordion_button_class = speaker.links.compact_blank.present? ? "accordion-button" : "accordion-button collapsed" %>
+        <%= button_tag "Social media links", type: "button", class: accordion_button_class, data: { bs_toggle: "collapse", bs_target: "#collapseBody"} %>
+      </h2>
+      <% accordion_collapse_class = speaker.links.compact_blank.present? ? "accordion-collapse collapse show" : "accordion-collapse collapse" %>
+      <div id="collapseBody" class="<%= accordion_collapse_class %>" >
+        <div class="accordion-body">
+          <% Speaker::SOCIAL_MEDIA_LINKS.each do |link| %>
+            <div class="mb-3">
+              <div class="input-group">
+                <%= f.label link.to_sym, class: "input-group-text" %>
+                <%= f.text_field link.to_sym, class: "form-control", placeholder: "https://socialmedia.url", aria: { label: link.capitalize, describedby: link  } %>
+              </div>
+            </div>
+          <% end %>
+        </div>
+      </div>
+    </div>
+  </div>
+
+  <div class="actions text-center mb-4">
+    <%= f.submit "Save", class: "btn btn-primary btn-lg" %>
+    <%=link_to "Cancel", admin_speakers_path, class: "btn btn-outline-primary btn-lg" %>
+  </div>
+<% end %>
diff --git a/app/views/admin/speakers/edit.html.erb b/app/views/admin/speakers/edit.html.erb
new file mode 100644
index 00000000..9a1cce65
--- /dev/null
+++ b/app/views/admin/speakers/edit.html.erb
@@ -0,0 +1,6 @@
+<div class="row">
+  <div class="col col-lg-6">
+      <h1>Edit speaker</h1>
+      <%=render 'form', speaker: @speaker %>
+  </div>
+</div>
diff --git a/app/views/admin/speakers/index.html.erb b/app/views/admin/speakers/index.html.erb
new file mode 100644
index 00000000..26fb4c83
--- /dev/null
+++ b/app/views/admin/speakers/index.html.erb
@@ -0,0 +1,51 @@
+<h1 class="mb-4">Speaker List</h1>
+<div class="row">
+  <div class="col">
+    <%= link_to new_admin_speaker_path, class: "btn btn-primary btn-md mb-4" do %>
+      Create new Speaker
+      <i class="bi bi-plus fw-semibold"></i>
+    <% end %>
+  </div>
+</div>
+
+<div class="row" id="speakers">
+  <div class="col">
+    <div class="table-responsive">
+      <table class="table table-sm text-center table-hover">
+        <thead class="border-2">
+        <tr class="table-info">
+            <th scope="col" class="p-3">Id</th>
+            <th scope="col" class="p-3">Name</th>
+            <th scope="col" class="p-3">Bio</th>
+            <th scope="col" class="p-3">Tagline</th>
+            <th scope="col" class="p-3">Image</th>
+            <th scope="col" class="p-3">Links</th>
+            <th scope="col" class="p-3">Actions</th>
+        </tr>
+        </thead>
+        <tbody>
+
+          <% @speakers.each do |speaker| %>
+            <tr class="border-2">
+              <td class="p-3 align-middle"><%= speaker.id %></td>
+              <td class="p-3 align-middle"><%= speaker.name %></td>
+              <td class="p-3 align-middle text-start"><%= truncate_html(speaker.bio, length: 70) %></td>
+              <td class="p-3 align-middle"><%= speaker.tagline %></td>
+              <td class="p-3 align-middle">
+                <div class="imagen-container">
+                  <%=image_tag speaker.image_url, class:"rounded-circle object-fit-cover", height: 53, width: 53, alt:"Profile photo for #{speaker.name}" %>
+                </div>
+              </td>
+              <td class="p-3 align-middle">
+                <%= render "admin/speakers/index/speaker_links", speaker: speaker%>
+              </td>
+              <td class="p-3 align-middle">
+              <%= link_to "Edit", edit_admin_speaker_path(speaker) %>
+            </td>
+            </tr>
+          <% end %>
+        </tbody>
+      </table>
+    </div>
+  </div>
+</div>
diff --git a/app/views/admin/speakers/index/_speaker_links.html.erb b/app/views/admin/speakers/index/_speaker_links.html.erb
new file mode 100644
index 00000000..568e1aac
--- /dev/null
+++ b/app/views/admin/speakers/index/_speaker_links.html.erb
@@ -0,0 +1,25 @@
+<% if speaker.empty_links? %>
+  <span>-</span>
+<% else %>
+  <% if speaker.github.present? %>
+    <i class="bi bi-github" alt="github" title="github"></i>
+  <% end %>
+  <% if speaker.mastodon.present? %>
+    <i class="bi bi-mastodon" alt="mastodon" title="mastodon"></i>
+  <% end %>
+  <% if speaker.linkedin.present? %>
+    <i class="bi bi-linkedin" alt="linkedin" title="linkedin"></i>
+  <% end %>
+  <% if speaker.twitter.present? %>
+    <i class="bi bi-twitter" alt="twitter" title="twitter"></i>
+  <% end %>
+  <% if speaker.website.present? %>
+    <i class="bi bi-globe" alt="personal website" title="personal website"></i>
+  <% end %>
+  <% if speaker.other.present? %>
+    <i class="bi bi-link-45deg" alt="other social link" title="other social link"></i>
+  <% end %>
+<% end %>
+
+
+
diff --git a/app/views/admin/speakers/new.html.erb b/app/views/admin/speakers/new.html.erb
new file mode 100644
index 00000000..24c66f79
--- /dev/null
+++ b/app/views/admin/speakers/new.html.erb
@@ -0,0 +1,7 @@
+<div class="row">
+  <div class="col col-lg-6">
+    <h1>New Speaker</h1>
+
+    <%= render "form", speaker: @speaker %>
+  </div>
+</div>
diff --git a/app/views/admin/speakers/show.html.erb b/app/views/admin/speakers/show.html.erb
new file mode 100644
index 00000000..71136389
--- /dev/null
+++ b/app/views/admin/speakers/show.html.erb
@@ -0,0 +1,10 @@
+<p style="color: green"><%= notice %></p>
+
+<%= render @speaker %>
+
+<div>
+  <%= link_to "Edit this speaker", edit_admin_speaker_path(@speaker) %> |
+  <%= link_to "Back to speakers", admin_speakers_path %>
+
+  <%= button_to "Destroy this speaker", @speaker, method: :delete %>
+</div>
diff --git a/app/views/layouts/admin/_flashes.html.erb b/app/views/layouts/admin/_flashes.html.erb
index 39c35b92..d6958919 100644
--- a/app/views/layouts/admin/_flashes.html.erb
+++ b/app/views/layouts/admin/_flashes.html.erb
@@ -1,5 +1,5 @@
 <% if flash.any? %>
-  <div class="alert alert-info mt-5 position-fixed start-0 end-0 alert-dismissible fade show mx-1 mx-md-3" role="alert">
+  <div class="container-md alert alert-info mt-5 position-fixed start-0 end-0 alert-dismissible fade show mx-1 mx-md-3 mx-lg-auto z-3" role="alert">
     <ul class="list-unstyled mb-0">
       <% flash.each do |type, msg| %>
         <li><%= msg %></li>
diff --git a/app/views/layouts/admin/_form_errors.html.erb b/app/views/layouts/admin/_form_errors.html.erb
index 99c264a5..70081809 100644
--- a/app/views/layouts/admin/_form_errors.html.erb
+++ b/app/views/layouts/admin/_form_errors.html.erb
@@ -1,7 +1,7 @@
-<%- if object.errors.any?%>
-  <div class="alert alert-info mt-5 position-fixed start-0 end-0 alert-dismissible fade show mx-1 mx-md-3" role="alert">
-    <p class="font-normal"><%= pluralize(@event.errors.count, "error") %> prohibited this event from being saved:</p>
-    <ul class="list-unstyled mb-0">
+<%- if object.errors.any? %>
+  <div class="alert alert-danger z-3 alert-dismissible fade show" role="alert">
+    <p class="font-normal"><%= pluralize(object.errors.count, "error") %> prohibited this event from being saved:</p>
+    <ul class="mb-0">
       <% object.errors.full_messages.each do |msg| %>
         <li><%= msg %></li>
       <% end %>
diff --git a/app/views/layouts/admin/_header.html.erb b/app/views/layouts/admin/_header.html.erb
index 86bbe716..422dee7b 100644
--- a/app/views/layouts/admin/_header.html.erb
+++ b/app/views/layouts/admin/_header.html.erb
@@ -1,6 +1,6 @@
 <header class="fixed-top">
     <nav class="navbar navbar-expand bg-white">
-      <div class="container-fluid">
+      <div class="container-fluid container-md">
         <a class="navbar-brand" href="/admin/dashboard">
           <%= image_tag "logo.svg", width: 48, height: 48, alt: "WNB.rb", class: "mx-auto" %>
         </a>
@@ -8,12 +8,15 @@
           <span class="navbar-toggler-icon"></span>
         </button>
         <div class="collapse navbar-collapse" id="navbarSupportedContent">
-          <ul class="navbar-nav me-auto mb-0">
+          <ul class="navbar-nav nav-underline me-auto mb-0">
             <li class="nav-item">
-              <%= link_to "Events", admin_events_path, class: "nav-link text-end #{"active text-decoration-underline" if controller_name == "events"}" %>
+              <%= link_to "Events", admin_events_path, class: "nav-link text-end #{"active" if controller_name == "events"}" %>
+            </li>
+            <li class="nav-item">
+              <%= link_to "Speakers", admin_speakers_path, class: "nav-link text-end #{"active" if controller_name == "speakers"}" %>
             </li>
           </ul>
-        <%= button_to "Sign out", destroy_user_session_path, form_class: "text-end",class: "btn btn-sm btn-secondary mb-0", method: :delete %>
+        <%= button_to "Sign out", destroy_user_session_path, form_class: "text-end", class: "btn btn-sm btn-secondary mb-0", method: :delete %>
       </div>
     </nav>
 </header>
diff --git a/config/routes.rb b/config/routes.rb
index 84fb4ca2..ecc63a36 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -8,8 +8,10 @@
                sign_up: 'new',
                registration: 'register',
              }
+
   namespace :admin, constraints: { format: 'html' } do
     get 'dashboard', to: 'dashboard#show'
+    resources :speakers, only: %i[index new create edit update]
     resources :events, only: %i[index new create edit update destroy]
   end
 
diff --git a/spec/controllers/admin/speakers_controller_spec.rb b/spec/controllers/admin/speakers_controller_spec.rb
new file mode 100644
index 00000000..b376f020
--- /dev/null
+++ b/spec/controllers/admin/speakers_controller_spec.rb
@@ -0,0 +1,160 @@
+# frozen_string_literal: true
+
+require './spec/rails_helper'
+
+RSpec.describe Admin::SpeakersController, type: :controller do
+  describe 'GET #index' do
+    context 'when user is not authenticated' do
+      it 'redirects to login page' do
+        get :index
+        expect(response).to redirect_to(new_user_session_path)
+      end
+    end
+    context 'when user is not admin' do
+      it 'returns 401' do
+        sign_in create(:user)
+        get :index
+        expect(response).to have_http_status(:unauthorized)
+      end
+    end
+
+    context 'when user is admin' do
+      before { sign_in create(:user, :admin) }
+
+      it 'returns a success response' do
+        get :index
+        expect(response).to be_successful
+      end
+    end
+  end
+
+  describe 'GET #new' do
+    context 'when user is admin' do
+      before { sign_in create(:user, :admin) }
+
+      it 'returns a successful response' do
+        get :new
+        expect(response).to be_successful
+      end
+
+      it 'assigns a new speaker to @speaker' do
+        get :new
+        speaker = controller.instance_variable_get(:@speaker)
+        expect(speaker).to be_a(Speaker)
+        expect(speaker).to be_new_record
+      end
+    end
+  end
+
+  describe 'GET #edit' do
+    let(:speaker) { create(:speaker) }
+
+    context 'when user is not authenticated' do
+      it 'redirects to login page' do
+        get :edit, params: { id: 1 }
+        expect(response).to have_http_status(302)
+        expect(response).to redirect_to(new_user_session_path)
+      end
+    end
+
+    context 'when user is not admin' do
+      it 'returns 401' do
+        sign_in create(:user)
+        get :edit, params: { id: 1 }
+        expect(response).to have_http_status(:unauthorized)
+      end
+    end
+    context 'when user is admin' do
+      before { sign_in create(:user, :admin) }
+      let(:speaker) { FactoryBot.create(:speaker) }
+
+      it 'returns a success response' do
+        get :edit, params: { id: speaker.id }
+        expect(response).to be_successful
+      end
+
+      it 'sets the correct speaker instance variable' do
+        get :edit, params: { id: speaker.id }
+        expect(controller.instance_variable_get(:@speaker)).to eq(speaker)
+      end
+
+      it 'renders not found if speaker is not found' do
+        expect { get :edit, params: { id: 'nonexistent_id' } }.to raise_error(
+          ActiveRecord::RecordNotFound,
+        )
+      end
+    end
+  end
+
+  describe 'POST #create' do
+    let(:valid_params) { attributes_for(:speaker) }
+    let(:invalid_params) { attributes_for(:speaker, name: nil) }
+
+    context 'when user is not authenticated' do
+      it 'returns 302' do
+        post :create
+        expect(response).to have_http_status(302)
+        expect(response).to redirect_to(new_user_session_path)
+      end
+    end
+
+    context 'when user is admin' do
+      before { sign_in create(:user, :admin) }
+      it 'creates a new speaker with valid params' do
+        post :create, params: { speaker: valid_params }
+        expect(Speaker.last.name).to eq(valid_params[:name])
+        expect(response).to have_http_status(:redirect)
+      end
+
+      it 'does not create a new speaker with invalid params' do
+        expect { post :create, params: { speaker: invalid_params } }.not_to change(Speaker, :count)
+        expect(response).to have_http_status(:unprocessable_entity)
+      end
+    end
+  end
+
+  describe 'PATCH #update' do
+    context 'when user is not authenticated' do
+      it 'redirects to login page' do
+        put :update, params: { id: 1 }
+        expect(response).to have_http_status(302)
+        expect(response).to redirect_to(new_user_session_path)
+      end
+    end
+
+    context 'when user is not admin' do
+      before { sign_in create(:user) }
+      it 'returns 401' do
+        put :update, params: { id: 1 }
+        expect(response).to have_http_status(:unauthorized)
+      end
+    end
+
+    context 'when user is admin' do
+      before { sign_in create(:user, :admin) }
+
+      let(:speaker) { create(:speaker) }
+      let(:valid_params) { { name: 'Updated Name' } }
+      let(:invalid_params) { { name: nil } }
+
+      it 'returns a success response' do
+        put :update, params: { id: speaker.id, speaker: { name: 'new name' } }
+        expect(response).to redirect_to(edit_admin_speaker_path(speaker))
+        expect(speaker.reload.name).to eq('new name')
+      end
+
+      it 'updates the speaker with valid params' do
+        patch :update, params: { id: speaker.id, speaker: valid_params }
+        expect(speaker.reload.name).to eq('Updated Name')
+        expect(response).to have_http_status(:redirect)
+      end
+
+      it 'does not update the speaker with invalid params' do
+        original_name = speaker.name
+        patch :update, params: { id: speaker.id, speaker: invalid_params }
+        expect(speaker.reload.name).to eq(original_name)
+        expect(response).to have_http_status(:unprocessable_entity)
+      end
+    end
+  end
+end
diff --git a/spec/factories/speakers.rb b/spec/factories/speakers.rb
new file mode 100644
index 00000000..2ba09a00
--- /dev/null
+++ b/spec/factories/speakers.rb
@@ -0,0 +1,24 @@
+# frozen_string_literal: true
+FactoryBot.define do
+  factory :speaker do
+    name { Faker::Name.name }
+    bio { Faker::Lorem.paragraph }
+    tagline { Faker::Job.title }
+    image_url { Faker::Internet.url }
+  end
+
+  after(:build) { |_speaker| allow(HTTParty).to receive(:get).and_return(double(code: 200)) }
+
+  trait :with_invalid_links do
+    after(:build) { |_speaker| allow(HTTParty).to receive(:get).and_return(double(code: 401)) }
+    links { { notwitter: 'not-a-link', twitter: Faker::Internet.url } }
+  end
+
+  trait :with_valid_links do
+    links { { twitter: Faker::Internet.url, other: Faker::Internet.url } }
+  end
+
+  trait :with_empty_links do
+    links { {} }
+  end
+end
diff --git a/spec/models/speaker_spec.rb b/spec/models/speaker_spec.rb
index 1d4412bb..695e2154 100644
--- a/spec/models/speaker_spec.rb
+++ b/spec/models/speaker_spec.rb
@@ -13,4 +13,53 @@
     it { is_expected.to validate_presence_of(:bio) }
     it { is_expected.to validate_presence_of(:image_url) }
   end
-end
\ No newline at end of file
+
+  describe 'scopes' do
+    it 'orders speakers by name' do
+      speaker_b = create(:speaker, name: 'B Speaker')
+      speaker_a = create(:speaker, name: 'A Speaker')
+      expect(Speaker.ordered_by_name).to eq([speaker_a, speaker_b])
+    end
+  end
+  context 'when links are not present' do
+    it 'validates the social media links' do
+      speaker = build(:speaker)
+      speaker1 = build(:speaker, :with_empty_links)
+
+      expect(speaker).to be_valid
+      expect(speaker1).to be_valid
+    end
+  end
+
+  context 'when links are present' do
+    describe 'with valid links' do
+      it 'validates the social media links' do
+        speaker = build(:speaker, :with_valid_links)
+
+        expect(speaker).to be_valid
+      end
+    end
+
+    describe 'with invalid links' do
+      it 'validates the social media links' do
+        speaker = build(:speaker, :with_invalid_links)
+
+        expect(speaker).to be_invalid
+        expect(speaker.errors[:links]).to include('This social media is not allowed')
+      end
+    end
+  end
+
+  describe 'callbacks' do
+    describe '#format_links' do
+      let(:links) { { 'Twitter' => 'link_twitter', 'LinkedIn' => 'link_linkedIn' } }
+
+      it 'downcases the keys of the links' do
+        speaker = build(:speaker, links: links)
+
+        speaker.valid?
+        expect(speaker.links).to eq({ 'twitter' => 'link_twitter', 'linkedin' => 'link_linkedIn' })
+      end
+    end
+  end
+end
diff --git a/spec/rails_helper.rb b/spec/rails_helper.rb
index 3df076cb..9a1aed57 100644
--- a/spec/rails_helper.rb
+++ b/spec/rails_helper.rb
@@ -8,6 +8,12 @@
 abort('The Rails environment is running in production mode!') if Rails.env.production?
 require 'rspec/rails'
 # Add additional requires below this line. Rails is not loaded until this point!
+require 'factory_bot_rails'
+require 'rspec/mocks'
+
+# FactoryBot.factories.clear
+# FactoryBot.reload
+FactoryBot::SyntaxRunner.class_eval { include RSpec::Mocks::ExampleMethods }
 
 # Requires supporting ruby files with custom matchers and macros, etc, in
 # spec/support/ and its subdirectories. Files matching `spec/**/*_spec.rb` are
diff --git a/spec/system/managing_speakers_spec.rb b/spec/system/managing_speakers_spec.rb
new file mode 100644
index 00000000..e6cfeb47
--- /dev/null
+++ b/spec/system/managing_speakers_spec.rb
@@ -0,0 +1,45 @@
+# frozen_string_literal: true
+require 'rails_helper'
+
+RSpec.describe 'Managing speakers', type: :system do
+  let(:admin) { create(:user, :admin) }
+  let!(:speaker) { create(:speaker) }
+
+  before do
+    driven_by(:rack_test)
+
+    login_as admin
+    visit admin_speakers_path
+  end
+  context 'create a speaker' do
+    before { click_on 'Create new Speaker' }
+
+    it { expect(page).to have_text('New Speaker') }
+
+    describe 'with valid data' do
+      it 'creates an event' do
+        new_speaker = build(:speaker)
+
+        fill_in 'Name', with: new_speaker.name
+        fill_in 'Bio', with: new_speaker.bio
+        fill_in 'Tagline', with: new_speaker.tagline
+        fill_in 'Image link', with: new_speaker.image_url
+
+        click_on 'Save'
+
+        expect(page).to have_text('Speaker was successfully created.')
+        expect(page).to have_current_path(edit_admin_speaker_url(Speaker.last.id))
+      end
+    end
+
+    describe 'with invalid data' do
+      it 'shows errors' do
+        fill_in 'Name', with: ''
+
+        click_on 'Save'
+
+        expect(page).to have_text("Name can't be blank")
+      end
+    end
+  end
+end

From c6e76438c2b9b9ab0e48138c5ee322976514c108 Mon Sep 17 00:00:00 2001
From: mayra lucia navarro <luciagirasoles@gmail.com>
Date: Tue, 15 Aug 2023 02:26:09 -0500
Subject: [PATCH 3/4] Config custom error view to adapt to bootstrap

---
 config/initializers/custom_error.rb | 5 +++++
 1 file changed, 5 insertions(+)
 create mode 100644 config/initializers/custom_error.rb

diff --git a/config/initializers/custom_error.rb b/config/initializers/custom_error.rb
new file mode 100644
index 00000000..41980cee
--- /dev/null
+++ b/config/initializers/custom_error.rb
@@ -0,0 +1,5 @@
+# frozen_string_literal: true
+
+ActionView::Base.field_error_proc = proc do |html_tag, _instance|
+  html_tag.gsub('form-control', 'form-control is-invalid').html_safe
+end

From 4d4fd2c5200f823cb5d0ad6f5b2b9cf1f412a457 Mon Sep 17 00:00:00 2001
From: mayra lucia navarro <luciagirasoles@gmail.com>
Date: Tue, 15 Aug 2023 13:39:03 -0500
Subject: [PATCH 4/4] Fix lint errors

---
 .eslintrc.js                         |  5 +++++
 app/javascript/packs/hello_react.jsx | 24 ------------------------
 2 files changed, 5 insertions(+), 24 deletions(-)
 delete mode 100644 app/javascript/packs/hello_react.jsx

diff --git a/.eslintrc.js b/.eslintrc.js
index 79d0adfc..89b530f4 100644
--- a/.eslintrc.js
+++ b/.eslintrc.js
@@ -35,6 +35,11 @@ module.exports = {
             },
         ],
     },
+    settings: {
+        react: {
+            version: 'detect',
+        },
+    },
     overrides: [
         {
             files: [
diff --git a/app/javascript/packs/hello_react.jsx b/app/javascript/packs/hello_react.jsx
deleted file mode 100644
index e23eec2c..00000000
--- a/app/javascript/packs/hello_react.jsx
+++ /dev/null
@@ -1,24 +0,0 @@
-// Run this example by adding <%= javascript_pack_tag 'hello_react' %> to the head of your layout file,
-// like app/views/layouts/application.html.erb. All it does is render <div>Hello React</div> at the bottom
-// of the page.
-
-import React from 'react';
-import ReactDOM from 'react-dom';
-import PropTypes from 'prop-types';
-
-const Hello = (props) => <div>Hello {props.name}!</div>;
-
-Hello.defaultProps = {
-    name: 'David',
-};
-
-Hello.propTypes = {
-    name: PropTypes.string,
-};
-
-document.addEventListener('DOMContentLoaded', () => {
-    ReactDOM.render(
-        <Hello name="React" />,
-        document.body.appendChild(document.createElement('div')),
-    );
-});