Skip to content

  • Projects
  • Groups
  • Snippets
  • Help
  • This project
    • Loading...
  • Sign in / Register
V
VeNJOB
  • Overview
    • Overview
    • Details
    • Activity
    • Cycle Analytics
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Charts
  • Issues 0
    • Issues 0
    • List
    • Board
    • Labels
    • Milestones
  • Merge Requests 0
    • Merge Requests 0
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
    • Charts
  • Wiki
    • Wiki
  • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • Nguyen Hoang Mai Phuong
  • VeNJOB
  • Merge Requests
  • !7

Merged
Opened Aug 09, 2021 by Nguyen Hoang Mai Phuong@phuongnhm 
  • Report abuse
Report abuse

apply confirm page

×

Check out, review, and merge locally

Step 1. Fetch and check out the branch for this merge request

git fetch origin
git checkout -b apply-confirm-done-page origin/apply-confirm-done-page

Step 2. Review the changes locally

Step 3. Merge the branch and fix any conflicts that come up

git checkout master
git merge --no-ff apply-confirm-done-page

Step 4. Push the result of the merge to GitLab

git push origin master

Note that pushing to GitLab requires write access to this repository.

Tip: You can also checkout merge requests locally by following these guidelines.

  • Discussion 23
  • Commits 5
  • Pipelines 5
  • Changes 32
{{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
  • Nguyen Hoang Mai Phuong @phuongnhm

    added 1 commit

    • 33f82662 - apply confirm page

    Compare with previous version

    Aug 09, 2021

    added 1 commit

    • 33f82662 - apply confirm page

    Compare with previous version

    added 1 commit * 33f82662 - apply confirm page [Compare with previous version](https://gitlab.zigexn.vn/phuongnhm/VeNJOB/merge_requests/7/diffs?diff_id=5395&start_sha=0b7a5bba1f289aeba77e07ee9c9551c3cb819f98)
    Toggle commit list
  • Nguyen Hoang Mai Phuong @phuongnhm

    added 1 commit

    • b4c15816 - fix apply/confirm/done page

    Compare with previous version

    Aug 10, 2021

    added 1 commit

    • b4c15816 - fix apply/confirm/done page

    Compare with previous version

    added 1 commit * b4c15816 - fix apply/confirm/done page [Compare with previous version](https://gitlab.zigexn.vn/phuongnhm/VeNJOB/merge_requests/7/diffs?diff_id=5399&start_sha=33f826627c53b105c439655a4e1a5e09b424da25)
    Toggle commit list
  • Thanh Hung Pham
    @hungpt started a discussion on an old version of the diff Aug 10, 2021
    Last updated by Nguyen Hoang Mai Phuong Aug 12, 2021
    app/controllers/apply_jobs_controller.rb 0 → 100644
    1 class ApplyJobsController < ApplicationController
    2 def new
    3 @apply_job = ApplyJob.new
    • Thanh Hung Pham @hungpt commented Aug 10, 2021
      Master

      @phuongnhm Chỗ này bữa anh có nói

      • Từ params[:job_id] em nên kiểm tra Job có tồn tại hay không nha?
      • Nếu không tồn tại nên redirect về root
      • Sau này làm chức năng User thì kiểm tra thêm User đã login chưa?
      @phuongnhm Chỗ này bữa anh có nói - Từ ` params[:job_id]` em nên kiểm tra Job có tồn tại hay không nha? - Nếu không tồn tại nên redirect về root - Sau này làm chức năng User thì kiểm tra thêm User đã login chưa?
    • Nguyen Hoang Mai Phuong @phuongnhm

      changed this line in version 4 of the diff

      Aug 12, 2021

      changed this line in version 4 of the diff

      changed this line in [version 4 of the diff](https://gitlab.zigexn.vn/phuongnhm/VeNJOB/merge_requests/7/diffs?diff_id=5412&start_sha=b4c1581699a4a510a64b218e78263eee5c6dc09a#1aa63f12b57deb09a31d654c4c38ec55a37edc67_3_3)
      Toggle commit list
    Please register or sign in to reply
  • Thanh Hung Pham
    @hungpt started a discussion on an old version of the diff Aug 10, 2021
    Last updated by Nguyen Hoang Mai Phuong Aug 12, 2021
    app/controllers/apply_jobs_controller.rb 0 → 100644
    1 class ApplyJobsController < ApplicationController
    2 def new
    3 @apply_job = ApplyJob.new
    4 @apply_job.job_id = params[:job_id]
    5 end
    6
    7 def confirm
    8 @apply_job = ApplyJob.new(apply_params)
    9 @apply_job.job_id = apply_params[:job_id]
    • Thanh Hung Pham @hungpt commented Aug 10, 2021
      Master

      @phuongnhm

      • Chỗ này em đã có job_id trong apply_params rồi. Nên không cần gán lại nha.
      • Em validate thêm bằng Model ở đây nha.
      • Nếu validate fail thì render về new và báo message cho người dùng biết
      @phuongnhm - Chỗ này em đã có `job_id` trong `apply_params` rồi. Nên không cần gán lại nha. - Em validate thêm bằng Model ở đây nha. - Nếu validate fail thì render về new và báo message cho người dùng biết
    • Nguyen Hoang Mai Phuong @phuongnhm

      changed this line in version 4 of the diff

      Aug 12, 2021

      changed this line in version 4 of the diff

      changed this line in [version 4 of the diff](https://gitlab.zigexn.vn/phuongnhm/VeNJOB/merge_requests/7/diffs?diff_id=5412&start_sha=b4c1581699a4a510a64b218e78263eee5c6dc09a#1aa63f12b57deb09a31d654c4c38ec55a37edc67_9_16)
      Toggle commit list
    Please register or sign in to reply
  • Thanh Hung Pham
    @hungpt started a discussion on an old version of the diff Aug 10, 2021
    Last updated by Nguyen Hoang Mai Phuong Aug 12, 2021
    app/controllers/apply_jobs_controller.rb 0 → 100644
    1 class ApplyJobsController < ApplicationController
    2 def new
    3 @apply_job = ApplyJob.new
    4 @apply_job.job_id = params[:job_id]
    5 end
    6
    7 def confirm
    8 @apply_job = ApplyJob.new(apply_params)
    9 @apply_job.job_id = apply_params[:job_id]
    10 end
    11
    12 def done
    13 @apply_job = ApplyJob.new(apply_params)
    14 @apply_job.user_id = User.find_by(id: 1).id
    15 @apply_job.job_id = apply_params[:job_id]
    • Thanh Hung Pham @hungpt commented Aug 10, 2021
      Master

      @phuongnhm

      • Chỗ này em đã có job_id trong apply_params rồi. Nên không cần gán lại nha.
      @phuongnhm - Chỗ này em đã có `job_id` trong `apply_params` rồi. Nên không cần gán lại nha.
    • Nguyen Hoang Mai Phuong @phuongnhm

      changed this line in version 4 of the diff

      Aug 12, 2021

      changed this line in version 4 of the diff

      changed this line in [version 4 of the diff](https://gitlab.zigexn.vn/phuongnhm/VeNJOB/merge_requests/7/diffs?diff_id=5412&start_sha=b4c1581699a4a510a64b218e78263eee5c6dc09a#1aa63f12b57deb09a31d654c4c38ec55a37edc67_15_28)
      Toggle commit list
    Please register or sign in to reply
  • Thanh Hung Pham
    @hungpt started a discussion on an old version of the diff Aug 10, 2021
    Last updated by Nguyen Hoang Mai Phuong Aug 12, 2021
    app/controllers/apply_jobs_controller.rb 0 → 100644
    1 class ApplyJobsController < ApplicationController
    2 def new
    3 @apply_job = ApplyJob.new
    4 @apply_job.job_id = params[:job_id]
    5 end
    6
    7 def confirm
    8 @apply_job = ApplyJob.new(apply_params)
    9 @apply_job.job_id = apply_params[:job_id]
    10 end
    11
    12 def done
    13 @apply_job = ApplyJob.new(apply_params)
    14 @apply_job.user_id = User.find_by(id: 1).id
    15 @apply_job.job_id = apply_params[:job_id]
    16 if @apply_job.save!
    • Thanh Hung Pham @hungpt commented Aug 10, 2021
      Master

      @phuongnhm Sao chỗ này dùng save! vậy em?

      @phuongnhm Sao chỗ này dùng `save!` vậy em?
    • Nguyen Hoang Mai Phuong @phuongnhm

      changed this line in version 4 of the diff

      Aug 12, 2021

      changed this line in version 4 of the diff

      changed this line in [version 4 of the diff](https://gitlab.zigexn.vn/phuongnhm/VeNJOB/merge_requests/7/diffs?diff_id=5412&start_sha=b4c1581699a4a510a64b218e78263eee5c6dc09a#1aa63f12b57deb09a31d654c4c38ec55a37edc67_16_28)
      Toggle commit list
    Please register or sign in to reply
  • Thanh Hung Pham
    @hungpt started a discussion on the diff Aug 10, 2021
    Resolved by Thanh Hung Pham Aug 12, 2021
    app/controllers/apply_jobs_controller.rb 0 → 100644
    5 end
    6
    7 def confirm
    8 @apply_job = ApplyJob.new(apply_params)
    9 @apply_job.job_id = apply_params[:job_id]
    10 end
    11
    12 def done
    13 @apply_job = ApplyJob.new(apply_params)
    14 @apply_job.user_id = User.find_by(id: 1).id
    15 @apply_job.job_id = apply_params[:job_id]
    16 if @apply_job.save!
    17 ApplyJobMailer.with(apply_job: @apply_job).create_apply.deliver_now
    18 flash.now[:success] = "You have applied successfully"
    19 else
    20 redirect_to root_url
    • Thanh Hung Pham @hungpt commented Aug 10, 2021
      Master

      @phuongnhm Trường hợp này lỗi, nên thông báo message gì cho người dùng biết nha em.

      Edited Aug 12, 2021
      @phuongnhm Trường hợp này lỗi, nên thông báo message gì cho người dùng biết nha em.
    Please register or sign in to reply
  • Thanh Hung Pham
    @hungpt started a discussion on an old version of the diff Aug 10, 2021
    Last updated by Nguyen Hoang Mai Phuong Aug 12, 2021
    app/mailers/application_mailer.rb
    1 1 class ApplicationMailer < ActionMailer::Base
    2 default from: 'from@example.com'
    2 default from: 'nhmphuong1005@gmail.com'
    • Thanh Hung Pham @hungpt commented Aug 10, 2021
      Master

      @phuongnhm Chỗ này nên để email của hệ thống. Ví dụ như info@venjob.com chẳn hạn

      @phuongnhm Chỗ này nên để email của hệ thống. Ví dụ như `info@venjob.com` chẳn hạn
    • Nguyen Hoang Mai Phuong @phuongnhm

      changed this line in version 4 of the diff

      Aug 12, 2021

      changed this line in version 4 of the diff

      changed this line in [version 4 of the diff](https://gitlab.zigexn.vn/phuongnhm/VeNJOB/merge_requests/7/diffs?diff_id=5412&start_sha=b4c1581699a4a510a64b218e78263eee5c6dc09a#329dcb56769718d2f61bf2e5f74ab58934386470_2_2)
      Toggle commit list
    Please register or sign in to reply
  • Thanh Hung Pham
    @hungpt started a discussion on an old version of the diff Aug 10, 2021
    Last updated by Nguyen Hoang Mai Phuong Aug 12, 2021
    app/models/apply_job.rb
    1 1 class ApplyJob < ApplicationRecord
    2 2 belongs_to :user
    3 3 belongs_to :job
    4 has_one_attached :cv
    5 validates :name, presence: true, length: { maximum: 200 }
    6 VALID_EMAIL_REGEX = /\A[\w+\-.]+@[a-z\d\-.]+\.[a-z]+\z/i
    7 validates :email, presence: true, length: { maximum: 255 }, format: { with: VALID_EMAIL_REGEX }
    8 validates :cv, attached: true, content_type: { in: 'application/pdf', message: 'must be a valid cv format' }
    • Thanh Hung Pham @hungpt commented Aug 10, 2021
      Master

      @phuongnhm Validate thêm limit size của file nữa nha em.

      @phuongnhm Validate thêm limit size của file nữa nha em.
    • Nguyen Hoang Mai Phuong @phuongnhm

      changed this line in version 4 of the diff

      Aug 12, 2021

      changed this line in version 4 of the diff

      changed this line in [version 4 of the diff](https://gitlab.zigexn.vn/phuongnhm/VeNJOB/merge_requests/7/diffs?diff_id=5412&start_sha=b4c1581699a4a510a64b218e78263eee5c6dc09a#062f142b54fd37c3e89f3ca1e264db59f9c0d9e3_8_8)
      Toggle commit list
    Please register or sign in to reply
  • Thanh Hung Pham
    @hungpt started a discussion on an old version of the diff Aug 10, 2021
    Last updated by Nguyen Hoang Mai Phuong Aug 12, 2021
    app/views/apply_jobs/confirm.html.slim 0 → 100644
    1 .ribbon
    • Thanh Hung Pham @hungpt commented Aug 10, 2021
      Master

      @phuongnhm Chỗ này sao không render lại ribbon em?

      .ribbon
        ul.progressbar
          li
            .span.indicator
            .span.title Apply
          li.active
            .span.indicator.2
            .span.title Confirm
          li
            .span.indicator.3
            .span.title Done
      • Suy nghĩ thêm solution để render cái active cho từng step tương ứng nha
      Edited Aug 10, 2021 by Thanh Hung Pham
      @phuongnhm Chỗ này sao không render lại `ribbon` em? ``` .ribbon ul.progressbar li .span.indicator .span.title Apply li.active .span.indicator.2 .span.title Confirm li .span.indicator.3 .span.title Done ``` - Suy nghĩ thêm solution để render cái `active` cho từng step tương ứng nha
    • Nguyen Hoang Mai Phuong @phuongnhm

      changed this line in version 4 of the diff

      Aug 12, 2021

      changed this line in version 4 of the diff

      changed this line in [version 4 of the diff](https://gitlab.zigexn.vn/phuongnhm/VeNJOB/merge_requests/7/diffs?diff_id=5412&start_sha=b4c1581699a4a510a64b218e78263eee5c6dc09a#ee021601d7525cf15feb7e790387562d9174a38a_1_1)
      Toggle commit list
    Please register or sign in to reply
  • Thanh Hung Pham
    @hungpt started a discussion on an old version of the diff Aug 10, 2021
    Last updated by Nguyen Hoang Mai Phuong Aug 12, 2021
    app/views/apply_jobs/confirm.html.slim 0 → 100644
    1 .ribbon
    2 ul.progressbar
    3 li
    4 .span.indicator
    5 .span.title Apply
    6 li.active
    7 .span.indicator.2
    8 .span.title Confirm
    9 li
    10 .span.indicator.3
    11 .span.title Done
    12 h2.text-center.p-4
    13 | CONFIRM INFORMATION
    14 .col-md-6.offset-md-3
    15 = form_with(model: @apply_job, url: '/done', local: true) do |f|
    • Thanh Hung Pham @hungpt commented Aug 10, 2021
      Master

      @phuongnhm url: '/done' Chỗ này mình nên dùng helpers có sẵn nha em xxxx_path

      @phuongnhm `url: '/done'` Chỗ này mình nên dùng helpers có sẵn nha em `xxxx_path`
    • Nguyen Hoang Mai Phuong @phuongnhm

      changed this line in version 4 of the diff

      Aug 12, 2021

      changed this line in version 4 of the diff

      changed this line in [version 4 of the diff](https://gitlab.zigexn.vn/phuongnhm/VeNJOB/merge_requests/7/diffs?diff_id=5412&start_sha=b4c1581699a4a510a64b218e78263eee5c6dc09a#ee021601d7525cf15feb7e790387562d9174a38a_15_4)
      Toggle commit list
    Please register or sign in to reply
  • Thanh Hung Pham
    @hungpt started a discussion on an old version of the diff Aug 10, 2021
    Last updated by Nguyen Hoang Mai Phuong Aug 12, 2021
    app/views/apply_jobs/confirm.html.slim 0 → 100644
    3 li
    4 .span.indicator
    5 .span.title Apply
    6 li.active
    7 .span.indicator.2
    8 .span.title Confirm
    9 li
    10 .span.indicator.3
    11 .span.title Done
    12 h2.text-center.p-4
    13 | CONFIRM INFORMATION
    14 .col-md-6.offset-md-3
    15 = form_with(model: @apply_job, url: '/done', local: true) do |f|
    16 = f.hidden_field :job_id
    17 = f.label :name
    18 = f.text_field :name, required: true, disabled: true,length: { maximum: 200 }, placeholder: "Type name...", class: 'form-control'
    • Thanh Hung Pham @hungpt commented Aug 10, 2021
      Master

      @phuongnhm Add space after ,

      • disabled: true,length: { maximum: 200 }
      • Tương tự chỗ email
      Edited Aug 10, 2021 by Thanh Hung Pham
      @phuongnhm Add space after `,` - `disabled: true,length: { maximum: 200 }` - Tương tự chỗ `email`
    • Nguyen Hoang Mai Phuong @phuongnhm

      changed this line in version 4 of the diff

      Aug 12, 2021

      changed this line in version 4 of the diff

      changed this line in [version 4 of the diff](https://gitlab.zigexn.vn/phuongnhm/VeNJOB/merge_requests/7/diffs?diff_id=5412&start_sha=b4c1581699a4a510a64b218e78263eee5c6dc09a#ee021601d7525cf15feb7e790387562d9174a38a_18_9)
      Toggle commit list
    Please register or sign in to reply
  • Thanh Hung Pham
    @hungpt started a discussion on an old version of the diff Aug 10, 2021
    Last updated by Nguyen Hoang Mai Phuong Aug 12, 2021
    app/views/apply_jobs/confirm.html.slim 0 → 100644
    4 .span.indicator
    5 .span.title Apply
    6 li.active
    7 .span.indicator.2
    8 .span.title Confirm
    9 li
    10 .span.indicator.3
    11 .span.title Done
    12 h2.text-center.p-4
    13 | CONFIRM INFORMATION
    14 .col-md-6.offset-md-3
    15 = form_with(model: @apply_job, url: '/done', local: true) do |f|
    16 = f.hidden_field :job_id
    17 = f.label :name
    18 = f.text_field :name, required: true, disabled: true,length: { maximum: 200 }, placeholder: "Type name...", class: 'form-control'
    19 = f.hidden_field :name, required: true, length: { maximum: 50 }
    • Thanh Hung Pham @hungpt commented Aug 10, 2021
      Master

      @phuongnhm hidden field mà, không cần validate nha em. Tương tự bên email

      @phuongnhm hidden field mà, không cần validate nha em. Tương tự bên `email`
    • Nguyen Hoang Mai Phuong @phuongnhm

      changed this line in version 4 of the diff

      Aug 12, 2021

      changed this line in version 4 of the diff

      changed this line in [version 4 of the diff](https://gitlab.zigexn.vn/phuongnhm/VeNJOB/merge_requests/7/diffs?diff_id=5412&start_sha=b4c1581699a4a510a64b218e78263eee5c6dc09a#ee021601d7525cf15feb7e790387562d9174a38a_19_9)
      Toggle commit list
    Please register or sign in to reply
  • Thanh Hung Pham
    @hungpt started a discussion on an old version of the diff Aug 10, 2021
    Last updated by Nguyen Hoang Mai Phuong Aug 12, 2021
    app/views/apply_jobs/done.html.slim 0 → 100644
    1 .ribbon.py-4
    • Thanh Hung Pham @hungpt commented Aug 10, 2021
      Master

      @phuongnhm Tương tự render ribbon lại nha em

      @phuongnhm Tương tự render `ribbon` lại nha em
    • Nguyen Hoang Mai Phuong @phuongnhm

      changed this line in version 4 of the diff

      Aug 12, 2021

      changed this line in version 4 of the diff

      changed this line in [version 4 of the diff](https://gitlab.zigexn.vn/phuongnhm/VeNJOB/merge_requests/7/diffs?diff_id=5412&start_sha=b4c1581699a4a510a64b218e78263eee5c6dc09a#747ef1556cfeb83de067b54653ae2b788e938816_1_1)
      Toggle commit list
    Please register or sign in to reply
  • Thanh Hung Pham
    @hungpt started a discussion on an old version of the diff Aug 10, 2021
    Last updated by Nguyen Hoang Mai Phuong Aug 12, 2021
    app/views/apply_jobs/done.html.slim 0 → 100644
    1 .ribbon.py-4
    2 ul.progressbar
    3 li
    4 .span.indicator
    5 .span.title Apply
    6 li
    7 .span.indicator.2
    8 .span.title Confirm
    9 li.active
    10 .span.indicator.3
    11 .span.title Done
    12 .container.py-4
    13 h3.py-3.text-center
    14 | Thank you for apply
    • Thanh Hung Pham @hungpt commented Aug 10, 2021
      Master

      @phuongnhm Miss requirement nha em. Thêm link Back to TOP page quay về trang chủ nha em

      @phuongnhm Miss requirement nha em. Thêm link `Back to TOP page` quay về trang chủ nha em
    • Nguyen Hoang Mai Phuong @phuongnhm

      changed this line in version 4 of the diff

      Aug 12, 2021

      changed this line in version 4 of the diff

      changed this line in [version 4 of the diff](https://gitlab.zigexn.vn/phuongnhm/VeNJOB/merge_requests/7/diffs?diff_id=5412&start_sha=b4c1581699a4a510a64b218e78263eee5c6dc09a#747ef1556cfeb83de067b54653ae2b788e938816_14_4)
      Toggle commit list
    Please register or sign in to reply
  • Thanh Hung Pham
    @hungpt started a discussion on an old version of the diff Aug 10, 2021
    Last updated by Nguyen Hoang Mai Phuong Aug 12, 2021
    app/views/apply_jobs/new.html.slim 0 → 100644
    1 .ribbon
    • Thanh Hung Pham @hungpt commented Aug 10, 2021
      Master

      @phuongnhm Tương tự render ribbon lại nha em

      @phuongnhm Tương tự render `ribbon` lại nha em
    • Nguyen Hoang Mai Phuong @phuongnhm

      changed this line in version 4 of the diff

      Aug 12, 2021

      changed this line in version 4 of the diff

      changed this line in [version 4 of the diff](https://gitlab.zigexn.vn/phuongnhm/VeNJOB/merge_requests/7/diffs?diff_id=5412&start_sha=b4c1581699a4a510a64b218e78263eee5c6dc09a#fb86ba2efbeebb73da8f586ce89c0d9913c3ad3e_1_1)
      Toggle commit list
    Please register or sign in to reply
  • Thanh Hung Pham
    @hungpt started a discussion on an old version of the diff Aug 10, 2021
    Last updated by Nguyen Hoang Mai Phuong Aug 12, 2021
    app/views/apply_jobs/confirm.html.slim 0 → 100644
    14 .col-md-6.offset-md-3
    15 = form_with(model: @apply_job, url: '/done', local: true) do |f|
    16 = f.hidden_field :job_id
    17 = f.label :name
    18 = f.text_field :name, required: true, disabled: true,length: { maximum: 200 }, placeholder: "Type name...", class: 'form-control'
    19 = f.hidden_field :name, required: true, length: { maximum: 50 }
    20
    21 = f.label :email
    22 = f.text_field :email, required: true, disabled: true,length: { maximum: 255 }, format: { with: ApplyJob::VALID_EMAIL_REGEX}, placeholder: "Type email...", class: 'form-control'
    23 = f.hidden_field :email, required: true, length: { maximum: 255 }, format: { with: ApplyJob::VALID_EMAIL_REGEX}
    24 p.my-1
    25 = f.label :cv, "CV"
    26 p
    27 = f.file_field :cv, required: true, accept: "application/pdf"
    28 .span.p-4.text-center
    29 = link_to "Back", apply_path, class: "btn btn-primary btn-lg btn-space"
    • Thanh Hung Pham @hungpt commented Aug 10, 2021
      Master

      @phuongnhm Em test xem Back có giữ được thông tin người dùng đã nhập không?

      @phuongnhm Em test xem `Back` có giữ được thông tin người dùng đã nhập không?
    • Nguyen Hoang Mai Phuong @phuongnhm

      changed this line in version 4 of the diff

      Aug 12, 2021

      changed this line in version 4 of the diff

      changed this line in [version 4 of the diff](https://gitlab.zigexn.vn/phuongnhm/VeNJOB/merge_requests/7/diffs?diff_id=5412&start_sha=b4c1581699a4a510a64b218e78263eee5c6dc09a#ee021601d7525cf15feb7e790387562d9174a38a_29_20)
      Toggle commit list
    Please register or sign in to reply
  • Thanh Hung Pham
    @hungpt started a discussion on an old version of the diff Aug 10, 2021
    Last updated by Nguyen Hoang Mai Phuong Aug 12, 2021
    app/views/apply_jobs/new.html.slim 0 → 100644
    2 ul.progressbar
    3 li.active
    4 .span.indicator
    5 .span.title Apply
    6 li
    7 .span.indicator.2
    8 .span.title Confirm
    9 li
    10 .span.indicator.3
    11 .span.title Done
    12 h2.text-center.p-4
    13 | APPLY INFORMATION
    14 .col-md-6.offset-md-3
    15 = form_with(model: @apply_job, url: '/confirm', local: true) do |f|
    16 = f.hidden_field :job_id
    17 = f.text_field :name, required: true,length: { maximum: 200 }, placeholder: "Type name...", class: 'form-control'
    • Thanh Hung Pham @hungpt commented Aug 10, 2021
      Master

      @phuongnhm Add space after ,

      • required: true,length:
      @phuongnhm Add space after `,` - `required: true,length:`
    • Thanh Hung Pham @hungpt commented Aug 12, 2021
      Master

      @phuongnhm Chỗ này chưa sửa hả em?

      @phuongnhm Chỗ này chưa sửa hả em?
    • Nguyen Hoang Mai Phuong @phuongnhm

      changed this line in version 5 of the diff

      Aug 12, 2021

      changed this line in version 5 of the diff

      changed this line in [version 5 of the diff](https://gitlab.zigexn.vn/phuongnhm/VeNJOB/merge_requests/7/diffs?diff_id=5414&start_sha=f17e21bc5dde2ef1641d88ebcf79ec4522290daf#fb86ba2efbeebb73da8f586ce89c0d9913c3ad3e_9_9)
      Toggle commit list
    Please register or sign in to reply
  • Thanh Hung Pham
    @hungpt started a discussion on the diff Aug 10, 2021
    Resolved by Thanh Hung Pham Aug 12, 2021
    app/views/jobs/_search.html.slim
    1 1 .search.p-3.offset-md-2
    2 = form_with(url: '/jobs', method: 'get', local: true) do
    2 = form_with(url: '/jobs', local: true) do
    • Thanh Hung Pham @hungpt commented Aug 10, 2021
      Master

      @phuongnhm Tương tự, chỗ này dùng helpers có sẵn nha em xxxx_path

      Edited Aug 12, 2021
      @phuongnhm Tương tự, chỗ này dùng helpers có sẵn nha em `xxxx_path`
    • Thanh Hung Pham @hungpt commented Aug 12, 2021
      Master

      @phuongnhm chỗ này chưa sửa hả em?

      Edited Aug 12, 2021
      @phuongnhm chỗ này chưa sửa hả em?
    Please register or sign in to reply
  • Thanh Hung Pham
    @hungpt started a discussion on the diff Aug 10, 2021
    Resolved by Thanh Hung Pham Aug 12, 2021
    config/credentials.yml.enc
    1 DrY0/AhPzESn8oDFDPTUQsBNEvk49s13rSSpqNIGN57qdKMPNAsXwCKW3A/cAboje8Y5R0GxXkKFdKYWIK5ugq7ohO9d08CTuVeWhy4IATS1aJdy1GRYLg3F31k+X8Mtcf0J9mQmjBMLg9y0d69s6edaTpwtjvjkirXvbGuF2QxkWQv4vEXCqzCnQzrUbP2MIR8Oe4n0CfEZHeQVjY9tb2aFho+0hZosrSNzVVWnt4OGITjD6joHx31gJjshbWsSgumId0TkAdj2xBTUuMYwOk/21o7cBa974fzDuLuDjIZxm0joGOnsakj7j6db0xfkCG72dqXLOMIS2L6CH+skSHGMqWQEORMVm6qJsa9JyqYofIaFz4rCVqdrfG7p9GGBQHmxb4xuMFklfGLq3D9U9CMsY1/DRt7+JGky--sjw58TA1XGrY/KWE--lvwZbxtAhxp+E6KZw+vMjQ==
    • Thanh Hung Pham @hungpt commented Aug 10, 2021
      Master

      @phuongnhm File này không nên push lên nha em

      • Cho vào .gitignore nha em.
      Edited Aug 12, 2021
      @phuongnhm File này không nên push lên nha em - Cho vào `.gitignore` nha em.
    Please register or sign in to reply
  • Thanh Hung Pham
    @hungpt started a discussion on the diff Aug 10, 2021
    Resolved by Thanh Hung Pham Aug 12, 2021
    config/environments/development.rb
    33 33 # Store uploaded files on the local file system (see config/storage.yml for options).
    34 34 config.active_storage.service = :local
    35 35
    36 config.action_mailer.perform_deliveries = true
    36 37 # Don't care if the mailer can't send.
    37 38 config.action_mailer.raise_delivery_errors = false
    39 config.action_mailer.delivery_method = :smtp
    • Thanh Hung Pham @hungpt commented Aug 10, 2021
      Master

      @phuongnhm Chưa có config cho môi trường production hả em?

      Edited Aug 12, 2021
      @phuongnhm Chưa có config cho môi trường `production` hả em?
    Please register or sign in to reply
  • Nguyen Hoang Mai Phuong @phuongnhm

    added 1 commit

    • f17e21bc - fix progress bar and add session for input field

    Compare with previous version

    Aug 12, 2021

    added 1 commit

    • f17e21bc - fix progress bar and add session for input field

    Compare with previous version

    added 1 commit * f17e21bc - fix progress bar and add session for input field [Compare with previous version](https://gitlab.zigexn.vn/phuongnhm/VeNJOB/merge_requests/7/diffs?diff_id=5412&start_sha=b4c1581699a4a510a64b218e78263eee5c6dc09a)
    Toggle commit list
  • Thanh Hung Pham
    @hungpt started a discussion on an old version of the diff Aug 12, 2021
    Last updated by Nguyen Hoang Mai Phuong Aug 12, 2021
    db/migrate/20210809075523_remove_cv_from_apply_jobs.rb 0 → 100644
    1 class RemoveCvFromApplyJobs < ActiveRecord::Migration[6.1]
    2 def change
    3 remove_column :apply_jobs, :cv, :binary
    • Thanh Hung Pham @hungpt commented Aug 12, 2021
      Master

      @phuongnhm Chỗ này không ảnh hưởng nhiều. Nhưng remove_column không cần kiểu dữ liệu nha em.

      @phuongnhm Chỗ này không ảnh hưởng nhiều. Nhưng `remove_column` không cần kiểu dữ liệu nha em.
    • Nguyen Hoang Mai Phuong @phuongnhm

      changed this line in version 5 of the diff

      Aug 12, 2021

      changed this line in version 5 of the diff

      changed this line in [version 5 of the diff](https://gitlab.zigexn.vn/phuongnhm/VeNJOB/merge_requests/7/diffs?diff_id=5414&start_sha=f17e21bc5dde2ef1641d88ebcf79ec4522290daf#f5db588a83db243088e2ac1d2b1b4ce3575c3b32_3_3)
      Toggle commit list
    Please register or sign in to reply
  • Thanh Hung Pham
    @hungpt started a discussion on the diff Aug 12, 2021
    Resolved by Thanh Hung Pham Aug 12, 2021
    app/helpers/application_helper.rb
    4 4
    5 5 input.strftime('%d/%m/%Y')
    6 6 end
    7
    8 def class_active(link_path)
    9 return unless current_page?(link_path)
    10
    11 'active'
    12 end
    13
    14 def current_page?(*paths)
    • Thanh Hung Pham @hungpt commented Aug 12, 2021
      Master

      @phuongnhm Method này refactor lại vậy cũng được nè em

      def current_page?(*paths)
        paths.include?(request.path)
      end
      Edited Aug 12, 2021
      @phuongnhm Method này refactor lại vậy cũng được nè em ``` def current_page?(*paths) paths.include?(request.path) end ```
    Please register or sign in to reply
  • Nguyen Hoang Mai Phuong @phuongnhm

    added 1 commit

    • 7c86d4ed - fix style code in view and refactor code

    Compare with previous version

    Aug 12, 2021

    added 1 commit

    • 7c86d4ed - fix style code in view and refactor code

    Compare with previous version

    added 1 commit * 7c86d4ed - fix style code in view and refactor code [Compare with previous version](https://gitlab.zigexn.vn/phuongnhm/VeNJOB/merge_requests/7/diffs?diff_id=5414&start_sha=f17e21bc5dde2ef1641d88ebcf79ec4522290daf)
    Toggle commit list
  • Thanh Hung Pham @hungpt

    mentioned in commit 8ca02714

    Aug 12, 2021

    mentioned in commit 8ca02714

    mentioned in commit 8ca027145cdc9b8b0ec5b234ceb22fdf6e0ff3ca
    Toggle commit list
  • Thanh Hung Pham @hungpt

    merged

    Aug 12, 2021

    merged

    merged
    Toggle commit list
  • Write
  • Preview
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or sign in to comment
Assignee
No assignee
Assign to
None
Milestone
None
Assign milestone
Time tracking
2
2 participants
Reference: phuongnhm/VeNJOB!7
×

Revert this merge request

Switch branch
Cancel
A new branch will be created in your fork and a new merge request will be started.
×

Cherry-pick this merge request

Switch branch
Cancel
A new branch will be created in your fork and a new merge request will be started.