fix mentor's comments

app/controllers/job_applieds_controller.rb

@@ -21,7 +21,7 @@ class JobAppliedsController < ApplicationController
   end
   end
 
 
   def show
   def show
-    @jobs = Job.applied_job(current_user.id).page(params[:page]).per(Job::LIMIT_PAGE)
+    @users = current_user.job_applieds.order("job_applieds.updated_at DESC").page(params[:page]).per(Job::LIMIT_PAGE)
   end
   end
 
 
   def confirmation
   def confirmation

app/models/job.rb

@@ -19,9 +19,6 @@ class Job < ApplicationRecord
 
 
   LIMIT_PAGE = 20
   LIMIT_PAGE = 20
 
 
-  scope :applied_job, ->(user_id) { joins(:job_applieds)
-                                    .where("job_applieds.user_id = #{user_id}")
-                                    .order("job_applieds.updated_at DESC") }
   scope :limit_job, -> { includes(:cities, :company).order(created_at: :desc).limit(5) }
   scope :limit_job, -> { includes(:cities, :company).order(created_at: :desc).limit(5) }
 
 
   def company_name
   def company_name

app/models/job_applied.rb

@@ -9,4 +9,5 @@ class JobApplied < ApplicationRecord
   VALID_EMAIL_REGEX = /\A[\w+\-.]+@[a-z\d\-]+(?:\.[a-z\d\-]+)*\.[a-z]+\z/i
   VALID_EMAIL_REGEX = /\A[\w+\-.]+@[a-z\d\-]+(?:\.[a-z\d\-]+)*\.[a-z]+\z/i
   validates :email, presence: true, length: { maximum: 200 }, format: { with: VALID_EMAIL_REGEX }
   validates :email, presence: true, length: { maximum: 200 }, format: { with: VALID_EMAIL_REGEX }
   validates :cv_user, presence: true
   validates :cv_user, presence: true
+
 end
 end

app/views/job_applieds/_my_jobs.html.erb

 <div class="border border-dark rounded">
 <div class="border border-dark rounded">
   <div class="job-details">
   <div class="job-details">
     <div class="title">
     <div class="title">
-      <%= link_to job_detail_path(job.id) do %><strong>
-        <%= job.title %></strong>
+      <%= link_to job_detail_path(user.job.id) do %><strong>
+        <%= user.job.title %></strong>
       <% end %>
       <% end %>
     </div>
     </div>
     <div class="introduction">
     <div class="introduction">
-      <%= strip_tags(job.format_desc) %><br>
-      <%= link_to 'Read more..', job_detail_path(job.id) %>
+      <%= strip_tags(user.job.format_desc) %><br>
+      <%= link_to 'Read more..', job_detail_path(user.job.id) %>
     </div>
     </div>
     <div class="row">
     <div class="row">
       <div class="col-6">▼
       <div class="col-6">▼
-        <%= job.cities.map(&:name).join(' | ') %>
+        <%= user.job.cities.map(&:name).join(' | ') %>
       </div>
       </div>
       <div class="salary col-3">
       <div class="salary col-3">
-        Salary: <%= job.salary %>
+        Salary: <%= user.job.salary %>
       </div>
       </div>
       <div class="applied-at col-3">
       <div class="applied-at col-3">
         <strong>Applied at:</strong>
         <strong>Applied at:</strong>
-        <% job.job_applieds.where(user_id: current_user.id).each do |info| %>
-          <%= info.updated_at.strftime('%d/%m/%Y') %>
-        <% end %>
+        <%= user.updated_at.strftime('%d/%m/%Y') %>
       </div>
       </div>
     </div>
     </div>
   </div>
   </div>

app/views/job_applieds/show.html.erb

 <div class="container">
 <div class="container">
-  <% if @jobs.count > 0 %>
+  <% if @users.count > 0 %>
     <h1 class="text-center my-job-label">My Job</h1>
     <h1 class="text-center my-job-label">My Job</h1>
-    <%= paginate @jobs, outer_window: 2, window: 1 %>
-      <%= render partial: "job_applieds/my_jobs", collection: @jobs, as: :job %>
-    <%= paginate @jobs, outer_window: 2, window: 1 %>
+    <%= paginate @users, outer_window: 2, window: 1 %>
+      <%= render partial: "job_applieds/my_jobs", collection: @users, as: :user %>
+    <%= paginate @users, outer_window: 2, window: 1 %>
   <% else %>
   <% else %>
     <h1 class="let-apply">Let's apply job</h1>
     <h1 class="let-apply">Let's apply job</h1>
   <% end %>
   <% end %>