Implement style changes

This commit removes/adds spaces where needed, removes unnecessary comments
and changes <a> to link_to in the sidebar.

There are still some other <a> in the sidebar that I haven't changed yet
as other people are in the process of adding routes to these pages
(Topics and Languages).

Author: Oli0li

app/controllers/users_controller.rb

@@ -5,6 +5,10 @@ def index
     @users = User.all
   end
 
+  def new
+    @user = User.new
+  end
+
   def create
     @user = User.new(user_params)
 
@@ -17,10 +21,6 @@ def create
     end
   end
 
-
-  def new
-    @user = User.new
-  end
   def edit
   end
 
@@ -33,6 +33,7 @@ def update
       end
     end
   end
+
   def destroy
     @user.destroy!
 
@@ -42,6 +43,7 @@ def destroy
   end
 
   private
+
   def set_user
     @user = User.find(params.expect(:id))
   end

app/views/layouts/_sidebar.html.erb

@@ -1,4 +1,4 @@
-      
+
 <div id="sidebar">
   <div class="sidebar-wrapper active">
     <div class="sidebar-header position-relative">
@@ -72,16 +72,11 @@
           <% end %>
         </li>
 
-<<<<<<< HEAD
         <li class="sidebar-item">
-          <a href="application-chat.html" class="sidebar-link">
-=======
-        <li class="sidebar-item  ">
-          <a href="<%= users_path%>" class='sidebar-link'>
->>>>>>> add user crud action
+          <%= link_to users_path, class: 'sidebar-link' do %>
             <i class="bi bi-people"></i>
             <span>Users</span>
-          </a>
+          <% end %>
         </li>
       </ul>
     </div>

spec/requests/users_spec.rb

@@ -1,32 +1,14 @@
 require "rails_helper"
 
-# This spec was generated by rspec-rails when you ran the scaffold generator.
-# It demonstrates how one might use RSpec to test the controller code that
-# was generated by Rails when you ran the scaffold generator.
-#
-# It assumes that the implementation code is generated by the rails scaffold
-# generator. If you are using any extension libraries to generate different
-# controller code, this generated spec may or may not pass.
-#
-# It only uses APIs available in rails and/or rspec-rails. There are a number
-# of tools you can use to make these specs even more expressive, but we're
-# sticking to rails and rspec-rails APIs to keep things simple and stable.
-
 RSpec.describe "/users", type: :request do
   let(:user) { create(:user) }
-
+  let(:valid_attributes) { { email: "john@doe.com", password: "123" } }
+  let(:invalid_attributes) { { email: "" } }
 
   before do
     sign_in(user)
   end
 
-  # This should return the minimal set of attributes required to create a valid
-  # User. As you add validations to User, be sure to
-  # adjust the attributes here as well.
-  let(:valid_attributes) { { email: "john@doe.com", password: "123" } }
-
-  let(:invalid_attributes) { { email: "" } }
-
   describe "GET /index" do
     it "renders a successful response" do
       user = FactoryBot.create(:user)