-
Notifications
You must be signed in to change notification settings - Fork 25
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
Show vacancy to moderators #51
base: master
Are you sure you want to change the base?
Conversation
@@ -2,5 +2,7 @@ | |||
|
|||
root to: 'dashboard#index' | |||
|
|||
get 'vacancy/:id', to: 'dashboard#show' |
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.
Не очень уверен в правильности такого решения.
Просто как самая простая опция.
@@ -0,0 +1,42 @@ | |||
li.list-group-item |
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.
Опять же, вынес в партиал и использовал таким образом как самый простой вариант.
|
||
private | ||
|
||
def for_modertion |
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.
for_moderation
, typo :-)
|
||
it { expect(subject).to redirect_to '/moderation' } | ||
end | ||
|
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.
Extra newline
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.
@gruz0 Extra new lines are in every file, so I guess that should be considered a style guide for this project for consistency.
No description provided.