Skip to content

  • Projects
  • Groups
  • Snippets
  • Help
  • This project
    • Loading...
  • Sign in / Register
V
venjobs_app
  • 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
  • Quang Vinh Nguyen
  • venjobs_app
  • Merge Requests
  • !6

Merged
Opened Jun 20, 2018 by Quang Vinh Nguyen@vinhnq 
  • Report abuse
Report abuse

Add job controller

×

Check out, review, and merge locally

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

git fetch origin
git checkout -b add-job-controller origin/add-job-controller

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 add-job-controller

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 2
  • Commits 4
  • Changes 58
{{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
  • Van Toan Vu
    @toanvv started a discussion on the diff Jun 20, 2018
    app/controllers/static_pages_controller.rb 0 → 100644
    1 class StaticPagesController < ApplicationController
    2 def home
    3 @jobs = Job.all.order(updated_at: :desc).take(5)
    • Van Toan Vu @toanvv commented Jun 20, 2018
      Master

      try to use solr sort or Rails.cache.fetch to improve performance

      try to use solr sort or Rails.cache.fetch to improve performance
    Please register or sign in to reply
  • Van Toan Vu
    @toanvv started a discussion on the diff Jun 20, 2018
    app/controllers/jobs_controller.rb 0 → 100644
    12 def city
    13 @cities = City.all.select { |city| city.jobs.any? }
    14 end
    15
    16 def jobs_in_city
    17 @city = City.find_by(slug: params[:slug])
    18 @jobs = @city.jobs.page(params[:page])
    19 end
    20
    21 def home
    22 @jobs = Job.all.order(updated_at: :desc).take(5)
    23 @cities = City.all.select{ |city| city.jobs.any? }.take(8)
    24 end
    25
    26 def get_jobs_rsolr(title = '*')
    27 solr = RSolr.connect url: 'http://localhost:8983/solr/gettingstarted/'
    • Van Toan Vu @toanvv commented Jun 20, 2018
      Master

      write a class (recommend put on lib/ or services/) to reuse think about design class when search for

      • keyword
      • occupations
      • cities
      Edited Jun 20, 2018 by Van Toan Vu
      write a class (recommend put on lib/ or services/) to reuse think about design class when search for - keyword - occupations - cities
    Please register or sign in to reply
  • Quang Vinh Nguyen @vinhnq

    mentioned in commit 9f94ec6d

    Jun 20, 2018

    mentioned in commit 9f94ec6d

    mentioned in commit 9f94ec6df37e84dc14abb5ddc0520b5ee9af3891
    Toggle commit list
  • Quang Vinh Nguyen @vinhnq

    merged

    Jun 20, 2018

    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: vinhnq/venjobs_app!6
×

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.