-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
added job title functionality #27
base: master
Are you sure you want to change the base?
Conversation
@@ -0,0 +1,17 @@ | |||
class JobTitleUsersController < ApplicationController | |||
def create | |||
@job_title = JobTitleUser.new(job_title_params) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use 2 (not 4) spaces for indentation.
@@ -44,7 +44,6 @@ gem 'jquery-ui-rails' | |||
gem 'american_date' | |||
|
|||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
delete another line here
@@ -0,0 +1,3 @@ | |||
// Place all the styles related to the vacation_time controller here. | |||
// They will automatically be included in application.css. | |||
// You can use Sass (SCSS) here: http://sass-lang.com/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what is this file for?
looks good |
@@ -0,0 +1,2 @@ | |||
module VacationTimeHelper |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This file is empty and should be removed.
No description provided.