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 1
    • Merge Requests 1
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
    • Charts
  • Wiki
    • Wiki
  • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • Đường Sỹ Hoàng
  • VenJob
  • Merge Requests
  • !5

Merged
Opened Dec 20, 2019 by Đường Sỹ Hoàng@hoangds 
  • Report abuse
Report abuse

Feature/city_list/Updated city list

-Added City list, added method reject in sort_top_cities class method.

-Added view for VietNam & International list.

-Added method where in City model

city_list

Edited Dec 24, 2019 by Đường Sỹ Hoàng
×

Check out, review, and merge locally

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

git fetch origin
git checkout -b feature/city_list origin/feature/city_list

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 feature/city_list

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 21
  • Commits 9
  • Changes 20
{{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
  • Đường Sỹ Hoàng @hoangds

    changed title from First commit to Feature/city_list/first commit

    Dec 20, 2019

    changed title from First commit to Feature/city_list/first commit

    changed title from **First commit** to **F{+eature/city_list/f+}irst commit**
    Toggle commit list
  • Đường Sỹ Hoàng @hoangds

    changed the description

    Dec 20, 2019

    changed the description

    changed the description
    Toggle commit list
  • Son Do Hong
    @sondh started a discussion on an old version of the diff Dec 20, 2019
    Last updated by Đường Sỹ Hoàng Dec 24, 2019
    app/controllers/top_controller.rb
    1 1 class TopController < ApplicationController
    2 2 def index
    3 @latest_jobs = Job.latest_jobs.take(Settings.top.job.limit)
    4 @top_cities = City.sort_top_cities.take(Settings.top.city.limit)
    5 @top_industries = Industry.sort_top_industries.take(Settings.top.industry.limit)
    3 @latest_jobs = Job.latest_jobs
    4 @top_cities = City.sort_top_cities
    5 @top_industries = Industry.sort_top_industries
    6 end
    7
    8 def cities
    • Son Do Hong @sondh commented Dec 20, 2019
      Master

      viết thành hàm index của city controller

      viết thành hàm index của city controller
    • Đường Sỹ Hoàng @hoangds

      changed this line in version 5 of the diff

      Dec 24, 2019

      changed this line in version 5 of the diff

      changed this line in [version 5 of the diff](https://gitlab.zigexn.vn/hoangds/VenJob/merge_requests/5/diffs?diff_id=4539&start_sha=9ab3a46385263543ec11369f095bbdcd021427ae#93acd18eb0d0191ba8ca8a8c5bce6ac5e1ef67a8_8_7)
      Toggle commit list
    Please register or sign in to reply
  • Son Do Hong
    @sondh started a discussion on an old version of the diff Dec 20, 2019
    Last updated by Đường Sỹ Hoàng Dec 24, 2019
    config/routes.rb
    1 1 Rails.application.routes.draw do
    2 2 root "top#index"
    3 get "/cities", to: "top#cities"
    • Son Do Hong @sondh commented Dec 20, 2019
      Master

      sử dụng resources cities

      sử dụng resources cities
    • Đường Sỹ Hoàng @hoangds

      changed this line in version 2 of the diff

      Dec 24, 2019

      changed this line in version 2 of the diff

      changed this line in [version 2 of the diff](https://gitlab.zigexn.vn/hoangds/VenJob/merge_requests/5/diffs?diff_id=4536&start_sha=152daa6cc410813d01c6f8e5c3a635b74f4d7033#e801163eb9182955dca3eb9817d59af634f6665d_3_3)
      Toggle commit list
    Please register or sign in to reply
  • Son Do Hong
    @sondh started a discussion on an old version of the diff Dec 20, 2019
    Last updated by Đường Sỹ Hoàng Dec 24, 2019
    app/views/top/cities.html.erb 0 → 100644
    1 <div class="center jumbotron">
    2 <head>
    3 <title><%= yield(:title) %> Venjob</title>
    4 <meta name="viewport" content="width=device-width, initial-scale=1">
    • Son Do Hong @sondh commented Dec 20, 2019
      Master

      phần này viết vào header của layout

      phần này viết vào header của layout
    • Đường Sỹ Hoàng @hoangds

      changed this line in version 2 of the diff

      Dec 24, 2019

      changed this line in version 2 of the diff

      changed this line in [version 2 of the diff](https://gitlab.zigexn.vn/hoangds/VenJob/merge_requests/5/diffs?diff_id=4536&start_sha=152daa6cc410813d01c6f8e5c3a635b74f4d7033#149894f339d64c568cc3d6dc907de698e4498528_4_0)
      Toggle commit list
    Please register or sign in to reply
  • Son Do Hong
    @sondh started a discussion on an old version of the diff Dec 20, 2019
    Last updated by Đường Sỹ Hoàng Dec 24, 2019
    app/models/city.rb
    3 3 has_many :jobs, through: :city_jobs
    4 4
    5 5 def self.sort_top_cities
    6 @cities ||= all.sort_by(&:jobs_count).reverse
    6 @cities ||= all.sort_by(&:jobs_count).reject{ |city| city.jobs_count ==0 }.reverse.take(Settings.top.city.limit)
    • Son Do Hong @sondh commented Dec 20, 2019
      Master

      sử dụng zero? khi so sánh với 0

      sử dụng `zero?` khi so sánh với 0
    • Đường Sỹ Hoàng @hoangds

      changed this line in version 2 of the diff

      Dec 24, 2019

      changed this line in version 2 of the diff

      changed this line in [version 2 of the diff](https://gitlab.zigexn.vn/hoangds/VenJob/merge_requests/5/diffs?diff_id=4536&start_sha=152daa6cc410813d01c6f8e5c3a635b74f4d7033#28495972e963898abf2919ffeaeb92fbdf49fb8e_6_6)
      Toggle commit list
    Please register or sign in to reply
  • Son Do Hong
    @sondh started a discussion on an old version of the diff Dec 20, 2019
    Last updated by Đường Sỹ Hoàng Dec 24, 2019
    app/controllers/top_controller.rb
    1 1 class TopController < ApplicationController
    2 2 def index
    3 @latest_jobs = Job.latest_jobs.take(Settings.top.job.limit)
    4 @top_cities = City.sort_top_cities.take(Settings.top.city.limit)
    5 @top_industries = Industry.sort_top_industries.take(Settings.top.industry.limit)
    3 @latest_jobs = Job.latest_jobs
    4 @top_cities = City.sort_top_cities
    5 @top_industries = Industry.sort_top_industries
    6 end
    7
    8 def cities
    9 @top_cities = City.sort_top_cities
    • Son Do Hong @sondh commented Dec 20, 2019
      Master

      tách thành 2 biến, 1 là city của VN, 1 là city của quốc tế

      tách thành 2 biến, 1 là city của VN, 1 là city của quốc tế
    • Đường Sỹ Hoàng @hoangds

      changed this line in version 5 of the diff

      Dec 24, 2019

      changed this line in version 5 of the diff

      changed this line in [version 5 of the diff](https://gitlab.zigexn.vn/hoangds/VenJob/merge_requests/5/diffs?diff_id=4539&start_sha=9ab3a46385263543ec11369f095bbdcd021427ae#93acd18eb0d0191ba8ca8a8c5bce6ac5e1ef67a8_9_7)
      Toggle commit list
    Please register or sign in to reply
  • Son Do Hong
    @sondh started a discussion on an old version of the diff Dec 20, 2019
    Last updated by Đường Sỹ Hoàng Dec 24, 2019
    app/views/cities/_international_list.html.erb 0 → 100644
    1 <div class="container">
    2 <div class="row justify-content-md-center">
    3 <div class="col-sm">
    4 </div>
    • Son Do Hong @sondh commented Dec 20, 2019
      Master

      lúc có data thì nó sẽ hiển thị như thế nào ở đây

      lúc có data thì nó sẽ hiển thị như thế nào ở đây
    • Đường Sỹ Hoàng @hoangds

      changed this line in version 2 of the diff

      Dec 24, 2019

      changed this line in version 2 of the diff

      changed this line in [version 2 of the diff](https://gitlab.zigexn.vn/hoangds/VenJob/merge_requests/5/diffs?diff_id=4536&start_sha=152daa6cc410813d01c6f8e5c3a635b74f4d7033#3142664a80253c80a7b66cb53b3d60c3f196ccf8_4_0)
      Toggle commit list
    Please register or sign in to reply
  • Son Do Hong
    @sondh started a discussion on an old version of the diff Dec 20, 2019
    Last updated by Đường Sỹ Hoàng Dec 24, 2019
    app/views/top/_job.html.erb
    1 1 <div class="row align-items-center">
    2 2 <div class="col">
    3 <%= job.title %>
    3 <div class="text-center"><%= job.title %></div>
    • Son Do Hong @sondh commented Dec 20, 2019
      Master

      thêm text-center vào div bên ngoài, k cần tạo thêm div

      thêm `text-center` vào div bên ngoài, k cần tạo thêm div
    • Đường Sỹ Hoàng @hoangds

      changed this line in version 2 of the diff

      Dec 24, 2019

      changed this line in version 2 of the diff

      changed this line in [version 2 of the diff](https://gitlab.zigexn.vn/hoangds/VenJob/merge_requests/5/diffs?diff_id=4536&start_sha=152daa6cc410813d01c6f8e5c3a635b74f4d7033#522fcd960c9017d73d7402a59f5227cb68e98110_3_2)
      Toggle commit list
    Please register or sign in to reply
  • Son Do Hong
    @sondh started a discussion on an old version of the diff Dec 20, 2019
    Last updated by Đường Sỹ Hoàng Dec 25, 2019
    app/views/industries/_industry.html.erb
    1 1 <div class="container">
    2 2 <div class="row justify-content-md-center">
    3 <div class="col-sm">
    4 <%= industry.name %>
    5 (<%= industry.jobs.count %>)
    3 <div class="col">
    4 <div class="text-center">
    • Son Do Hong @sondh commented Dec 20, 2019
      Master

      thêm text-center vào div bên ngoài, k cần tạo thêm div

      thêm `text-center` vào div bên ngoài, k cần tạo thêm div
    • Đường Sỹ Hoàng @hoangds

      changed this line in version 8 of the diff

      Dec 25, 2019

      changed this line in version 8 of the diff

      changed this line in [version 8 of the diff](https://gitlab.zigexn.vn/hoangds/VenJob/merge_requests/5/diffs?diff_id=4550&start_sha=22462b9f1d705f2153fbbb771f2bff9617c50ad4#759f2125c06eb65fc6da7108a5ebda0a4631cbcc_4_3)
      Toggle commit list
    Please register or sign in to reply
  • Son Do Hong
    @sondh started a discussion on an old version of the diff Dec 20, 2019
    Last updated by Đường Sỹ Hoàng Dec 25, 2019
    app/views/cities/_cities_list.html.erb 0 → 100644
    1 <div class="container">
    2 <div class="row justify-content-md-center">
    3 <div class="col-sm">
    4 <div class="text-center">
    • Son Do Hong @sondh commented Dec 20, 2019
      Master

      thêm text-center vào div bên ngoài, k cần tạo thêm div

      thêm `text-center` vào div bên ngoài, k cần tạo thêm div
    • Đường Sỹ Hoàng @hoangds

      changed this line in version 7 of the diff

      Dec 25, 2019

      changed this line in version 7 of the diff

      changed this line in [version 7 of the diff](https://gitlab.zigexn.vn/hoangds/VenJob/merge_requests/5/diffs?diff_id=4549&start_sha=07fcf73bfdf05c0941427fb9322db4e9bd1d4809#11369fe55e876c6e062c64b181a00566f8a90e6e_4_3)
      Toggle commit list
    Please register or sign in to reply
  • Son Do Hong
    @sondh started a discussion on an old version of the diff Dec 20, 2019
    Last updated by Đường Sỹ Hoàng Dec 24, 2019
    app/views/cities/_city.html.erb
    1 1 <div class="container">
    2 2 <div class="row justify-content-md-center">
    3 <div class="col-sm">
    4 <%= city.name %>
    5 (<%= city.jobs.count %>)
    3 <div class="col">
    4 <div class="text-center">
    • Son Do Hong @sondh commented Dec 20, 2019
      Master

      thêm text-center vào div bên ngoài, k cần tạo thêm div

      thêm `text-center` vào div bên ngoài, k cần tạo thêm div
    • Đường Sỹ Hoàng @hoangds

      changed this line in version 2 of the diff

      Dec 24, 2019

      changed this line in version 2 of the diff

      changed this line in [version 2 of the diff](https://gitlab.zigexn.vn/hoangds/VenJob/merge_requests/5/diffs?diff_id=4536&start_sha=152daa6cc410813d01c6f8e5c3a635b74f4d7033#140e9386497ae5ab399fa6c34c5e8d27cd92fd63_4_3)
      Toggle commit list
    Please register or sign in to reply
  • Đường Sỹ Hoàng @hoangds

    added 1 commit

    • 8aa62865 - Updated city list

    Compare with previous version

    Dec 24, 2019

    added 1 commit

    • 8aa62865 - Updated city list

    Compare with previous version

    added 1 commit * 8aa62865 - Updated city list [Compare with previous version](https://gitlab.zigexn.vn/hoangds/VenJob/merge_requests/5/diffs?diff_id=4536&start_sha=152daa6cc410813d01c6f8e5c3a635b74f4d7033)
    Toggle commit list
  • Đường Sỹ Hoàng @hoangds

    added 1 commit

    • 554c726c - Fixed indentation

    Compare with previous version

    Dec 24, 2019

    added 1 commit

    • 554c726c - Fixed indentation

    Compare with previous version

    added 1 commit * 554c726c - Fixed indentation [Compare with previous version](https://gitlab.zigexn.vn/hoangds/VenJob/merge_requests/5/diffs?diff_id=4537&start_sha=8aa62865f79368767b55c3bb782e4fe56160f437)
    Toggle commit list
  • Đường Sỹ Hoàng @hoangds

    changed title from Feature/city_list/first commit to Feature/city_list/Updated city list

    Dec 24, 2019

    changed title from Feature/city_list/first commit to Feature/city_list/Updated city list

    changed title from **Feature/city_list/{-first commi-}t** to **Feature/city_list/{+Updated city lis+}t**
    Toggle commit list
  • Đường Sỹ Hoàng @hoangds

    changed the description

    Dec 24, 2019

    changed the description

    changed the description
    Toggle commit list
  • Van Hau Le
    @haulv started a discussion on an old version of the diff Dec 24, 2019
    Last updated by Đường Sỹ Hoàng Dec 24, 2019
    app/controllers/cities_controller.rb 0 → 100644
    1 class CitiesController < ApplicationController
    2 def index
    3 @top_vn_cities = City.sort_top_vn_cities
    4 @top_inter_cities = City.sort_top_inter_cities
    5 end
    6
    7 def vn_cities
    8 @top_vn_cities = City.sort_top_vn_cities
    9 end
    10
    11 def inter_cities
    12 @top_inter_cities = City.sort_top_inter_cities
    13 end
    • Van Hau Le @haulv commented Dec 24, 2019
      Master

      @hoangds chỗ này code bị thừa vậy em? sao phải get tới 2 lần vậy?

      @hoangds chỗ này code bị thừa vậy em? sao phải get tới 2 lần vậy?
    • Đường Sỹ Hoàng @hoangds

      changed this line in version 4 of the diff

      Dec 24, 2019

      changed this line in version 4 of the diff

      changed this line in [version 4 of the diff](https://gitlab.zigexn.vn/hoangds/VenJob/merge_requests/5/diffs?diff_id=4538&start_sha=554c726c307a5a9d7c4287f1b5294e3736b950e7#9331f3d6803b80e59ba5cd648677b3c6869d2ac8_13_6)
      Toggle commit list
    Please register or sign in to reply
  • Đường Sỹ Hoàng @hoangds

    added 1 commit

    • 9ab3a463 - Removed duplicated method

    Compare with previous version

    Dec 24, 2019

    added 1 commit

    • 9ab3a463 - Removed duplicated method

    Compare with previous version

    added 1 commit * 9ab3a463 - Removed duplicated method [Compare with previous version](https://gitlab.zigexn.vn/hoangds/VenJob/merge_requests/5/diffs?diff_id=4538&start_sha=554c726c307a5a9d7c4287f1b5294e3736b950e7)
    Toggle commit list
  • Đường Sỹ Hoàng @hoangds

    added 1 commit

    • 4276fcb8 - Removed cities method

    Compare with previous version

    Dec 24, 2019

    added 1 commit

    • 4276fcb8 - Removed cities method

    Compare with previous version

    added 1 commit * 4276fcb8 - Removed cities method [Compare with previous version](https://gitlab.zigexn.vn/hoangds/VenJob/merge_requests/5/diffs?diff_id=4539&start_sha=9ab3a46385263543ec11369f095bbdcd021427ae)
    Toggle commit list
  • Son Do Hong
    @sondh started a discussion on an old version of the diff Dec 24, 2019
    Last updated by Đường Sỹ Hoàng Dec 24, 2019
    app/models/city.rb
    3 3 has_many :jobs, through: :city_jobs
    4 4
    5 5 def self.sort_top_cities
    6 @cities ||= all.sort_by(&:jobs_count).reverse
    6 @cities ||= all.sort_by(&:jobs_count).reject{ |city| city.jobs_count.zero? }.reverse.take(Settings.top.city.limit)
    7 end
    8
    9 def self.sort_top_vn_cities
    10 @vn_cities ||= City.where(region: 0).sort_by(&:jobs_count).reject{ |city| city.jobs_count.zero? }.reverse.take(Settings.top.city.limit)
    • Son Do Hong @sondh commented Dec 24, 2019
      Master

      where(region: 0) ko cần City

      `where(region: 0)` ko cần City
    • Đường Sỹ Hoàng @hoangds

      changed this line in version 6 of the diff

      Dec 24, 2019

      changed this line in version 6 of the diff

      changed this line in [version 6 of the diff](https://gitlab.zigexn.vn/hoangds/VenJob/merge_requests/5/diffs?diff_id=4540&start_sha=4276fcb818a842fe7c4ae622cef3d2b00d91f6d1#28495972e963898abf2919ffeaeb92fbdf49fb8e_10_10)
      Toggle commit list
    Please register or sign in to reply
  • Son Do Hong
    @sondh started a discussion on an old version of the diff Dec 24, 2019
    Last updated by Đường Sỹ Hoàng Dec 24, 2019
    test/controllers/industries_controller_test.rb 0 → 100644
    1 require 'test_helper'
    2
    3 class IndustriesControllerTest < ActionDispatch::IntegrationTest
    4 # test "the truth" do
    5 # assert true
    • Son Do Hong @sondh commented Dec 24, 2019
      Master

      xóa file này

      xóa file này
    • Đường Sỹ Hoàng @hoangds

      changed this line in version 6 of the diff

      Dec 24, 2019

      changed this line in version 6 of the diff

      changed this line in [version 6 of the diff](https://gitlab.zigexn.vn/hoangds/VenJob/merge_requests/5/diffs?diff_id=4540&start_sha=4276fcb818a842fe7c4ae622cef3d2b00d91f6d1#1d4e995e8b825a78514908d10d6964efa07f8a34_5_0)
      Toggle commit list
    Please register or sign in to reply
  • Son Do Hong
    @sondh started a discussion on an old version of the diff Dec 24, 2019
    Last updated by Đường Sỹ Hoàng Dec 24, 2019
    test/controllers/city_controller_test.rb 0 → 100644
    1 require 'test_helper'
    2
    3 class CityControllerTest < ActionDispatch::IntegrationTest
    • Son Do Hong @sondh commented Dec 24, 2019
      Master

      xóa file này

      xóa file này
    • Đường Sỹ Hoàng @hoangds

      changed this line in version 6 of the diff

      Dec 24, 2019

      changed this line in version 6 of the diff

      changed this line in [version 6 of the diff](https://gitlab.zigexn.vn/hoangds/VenJob/merge_requests/5/diffs?diff_id=4540&start_sha=4276fcb818a842fe7c4ae622cef3d2b00d91f6d1#6d8017a97944d66ef0696fb4ce23daa36e93fcad_3_0)
      Toggle commit list
    Please register or sign in to reply
  • Son Do Hong
    @sondh started a discussion on an old version of the diff Dec 24, 2019
    Last updated by Đường Sỹ Hoàng Dec 24, 2019
    app/assets/stylesheets/city.scss 0 → 100644
    1 // Place all the styles related to the city controller here.
    2 // They will automatically be included in application.css.
    • Son Do Hong @sondh commented Dec 24, 2019
      Master

      xóa file này

      xóa file này
    • Đường Sỹ Hoàng @hoangds

      changed this line in version 6 of the diff

      Dec 24, 2019

      changed this line in version 6 of the diff

      changed this line in [version 6 of the diff](https://gitlab.zigexn.vn/hoangds/VenJob/merge_requests/5/diffs?diff_id=4540&start_sha=4276fcb818a842fe7c4ae622cef3d2b00d91f6d1#4bd85ade3736fd9911d04d4e9900c0548c5d7edb_2_0)
      Toggle commit list
    Please register or sign in to reply
  • Son Do Hong
    @sondh started a discussion on an old version of the diff Dec 24, 2019
    Last updated by Đường Sỹ Hoàng Dec 24, 2019
    app/assets/stylesheets/industries.scss 0 → 100644
    1 // Place all the styles related to the Industries controller here.
    • Son Do Hong @sondh commented Dec 24, 2019
      Master

      xóa file này

      xóa file này
    • Đường Sỹ Hoàng @hoangds

      changed this line in version 6 of the diff

      Dec 24, 2019

      changed this line in version 6 of the diff

      changed this line in [version 6 of the diff](https://gitlab.zigexn.vn/hoangds/VenJob/merge_requests/5/diffs?diff_id=4540&start_sha=4276fcb818a842fe7c4ae622cef3d2b00d91f6d1#53be1692cad40d36c9e93f0ebcf6e0d987ad32d1_1_0)
      Toggle commit list
    Please register or sign in to reply
  • Đường Sỹ Hoàng @hoangds

    added 1 commit

    • 07fcf73b - Removed files

    Compare with previous version

    Dec 24, 2019

    added 1 commit

    • 07fcf73b - Removed files

    Compare with previous version

    added 1 commit * 07fcf73b - Removed files [Compare with previous version](https://gitlab.zigexn.vn/hoangds/VenJob/merge_requests/5/diffs?diff_id=4540&start_sha=4276fcb818a842fe7c4ae622cef3d2b00d91f6d1)
    Toggle commit list
  • Van Hau Le
    @haulv started a discussion on an old version of the diff Dec 24, 2019
    Last updated by Đường Sỹ Hoàng Dec 25, 2019
    app/models/city.rb
    3 3 has_many :jobs, through: :city_jobs
    4 4
    5 5 def self.sort_top_cities
    6 @cities ||= all.sort_by(&:jobs_count).reverse
    6 @cities ||= all.sort_by(&:jobs_count).reject{ |city| city.jobs_count.zero? }.reverse.take(Settings.top.city.limit)
    • Van Hau Le @haulv commented Dec 24, 2019
      Master

      @hoangds check lại chỗ này có đnag bị N+1 query ko nhé?

      @hoangds check lại chỗ này có đnag bị N+1 query ko nhé?
    • Đường Sỹ Hoàng @hoangds commented Dec 24, 2019
      Master

      @haulv Ở chỗ này em có tìm hiểu thì có thể dùng counter cache hoặc xài gem counter_culture để giải quyết vấn đề N+1 query thì theo anh, em nên sử dụng cách nào ạ

      @haulv Ở chỗ này em có tìm hiểu thì có thể dùng counter cache hoặc xài gem counter_culture để giải quyết vấn đề N+1 query thì theo anh, em nên sử dụng cách nào ạ
    • Đường Sỹ Hoàng @hoangds

      changed this line in version 7 of the diff

      Dec 25, 2019

      changed this line in version 7 of the diff

      changed this line in [version 7 of the diff](https://gitlab.zigexn.vn/hoangds/VenJob/merge_requests/5/diffs?diff_id=4549&start_sha=07fcf73bfdf05c0941427fb9322db4e9bd1d4809#28495972e963898abf2919ffeaeb92fbdf49fb8e_6_6)
      Toggle commit list
    Please register or sign in to reply
  • Van Hau Le
    @haulv started a discussion on an old version of the diff Dec 24, 2019
    Last updated by Đường Sỹ Hoàng Dec 25, 2019
    app/models/city.rb
    3 3 has_many :jobs, through: :city_jobs
    4 4
    5 5 def self.sort_top_cities
    6 @cities ||= all.sort_by(&:jobs_count).reverse
    6 @cities ||= all.sort_by(&:jobs_count).reject{ |city| city.jobs_count.zero? }.reverse.take(Settings.top.city.limit)
    7 end
    8
    9 def self.sort_top_vn_cities
    10 @vn_cities ||= where(region: 0).sort_by(&:jobs_count).reject{ |city| city.jobs_count.zero? }.reverse.take(Settings.top.city.limit)
    • Van Hau Le @haulv commented Dec 24, 2019
      Master

      @hoangds tương tự ở trên và bên dưới của sort_top_inter_cities và file industry.rb

      @hoangds tương tự ở trên và bên dưới của `sort_top_inter_cities` và file industry.rb
    • Đường Sỹ Hoàng @hoangds

      changed this line in version 7 of the diff

      Dec 25, 2019

      changed this line in version 7 of the diff

      changed this line in [version 7 of the diff](https://gitlab.zigexn.vn/hoangds/VenJob/merge_requests/5/diffs?diff_id=4549&start_sha=07fcf73bfdf05c0941427fb9322db4e9bd1d4809#28495972e963898abf2919ffeaeb92fbdf49fb8e_10_10)
      Toggle commit list
    Please register or sign in to reply
  • Đường Sỹ Hoàng @hoangds

    added 1 commit

    • 22462b9f - Fixed industry list

    Compare with previous version

    Dec 25, 2019

    added 1 commit

    • 22462b9f - Fixed industry list

    Compare with previous version

    added 1 commit * 22462b9f - Fixed industry list [Compare with previous version](https://gitlab.zigexn.vn/hoangds/VenJob/merge_requests/5/diffs?diff_id=4549&start_sha=07fcf73bfdf05c0941427fb9322db4e9bd1d4809)
    Toggle commit list
  • Đường Sỹ Hoàng @hoangds

    added 1 commit

    • 9f03ade7 - Removed files

    Compare with previous version

    Dec 25, 2019

    added 1 commit

    • 9f03ade7 - Removed files

    Compare with previous version

    added 1 commit * 9f03ade7 - Removed files [Compare with previous version](https://gitlab.zigexn.vn/hoangds/VenJob/merge_requests/5/diffs?diff_id=4550&start_sha=22462b9f1d705f2153fbbb771f2bff9617c50ad4)
    Toggle commit list
  • Son Do Hong
    @sondh started a discussion on an old version of the diff Dec 25, 2019
    Last updated by Đường Sỹ Hoàng Dec 25, 2019
    config/routes.rb
    1 1 Rails.application.routes.draw do
    2 2 root "top#index"
    3 resources :cities, only: :index
    4 resources :industries, only: :index
    5 resources :login, only: :index
    • Son Do Hong @sondh commented Dec 25, 2019
      Master

      remove

      remove
    • Đường Sỹ Hoàng @hoangds

      changed this line in version 9 of the diff

      Dec 25, 2019

      changed this line in version 9 of the diff

      changed this line in [version 9 of the diff](https://gitlab.zigexn.vn/hoangds/VenJob/merge_requests/5/diffs?diff_id=4551&start_sha=9f03ade73924e9ee65f143960b6ab746a59c9ad7#e801163eb9182955dca3eb9817d59af634f6665d_5_5)
      Toggle commit list
    Please register or sign in to reply
  • Son Do Hong
    @sondh started a discussion on an old version of the diff Dec 25, 2019
    Last updated by Đường Sỹ Hoàng Dec 25, 2019
    app/views/layouts/_header.html.erb
    3 3 <%= link_to image_tag("zigexn_logo.png", id: "logo", alt: "Venjob logo") %>
    4 4 <nav class="navbar navbar-light bg-light">
    5 5 <form class="form-inline">
    6 <button class="btn btn-outline-success" type="button">Login</button>
    6 <%= link_to "Login", login_index_path, class:"btn btn-outline-success" %>
    • Son Do Hong @sondh commented Dec 25, 2019
      Master

      để lại như cũ

      để lại như cũ
    • Đường Sỹ Hoàng @hoangds

      changed this line in version 9 of the diff

      Dec 25, 2019

      changed this line in version 9 of the diff

      changed this line in [version 9 of the diff](https://gitlab.zigexn.vn/hoangds/VenJob/merge_requests/5/diffs?diff_id=4551&start_sha=9f03ade73924e9ee65f143960b6ab746a59c9ad7#12e0ffa73350b6f8b572afab727eeab71ee1df63_6_6)
      Toggle commit list
    Please register or sign in to reply
  • Đường Sỹ Hoàng @hoangds

    added 1 commit

    • a861c741 - Removed files

    Compare with previous version

    Dec 25, 2019

    added 1 commit

    • a861c741 - Removed files

    Compare with previous version

    added 1 commit * a861c741 - Removed files [Compare with previous version](https://gitlab.zigexn.vn/hoangds/VenJob/merge_requests/5/diffs?diff_id=4551&start_sha=9f03ade73924e9ee65f143960b6ab746a59c9ad7)
    Toggle commit list
  • Son Do Hong @sondh

    mentioned in commit a89c829f

    Dec 25, 2019

    mentioned in commit a89c829f

    mentioned in commit a89c829fe7c7ca28c9362ca43028ff11012ebcc1
    Toggle commit list
  • Son Do Hong @sondh

    merged

    Dec 25, 2019

    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
Son Do Hong
Assignee
Son Do Hong @sondh
Assign to
None
Milestone
None
Assign milestone
Time tracking
3
3 participants
Reference: hoangds/VenJob!5
×

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.