-
Notifications
You must be signed in to change notification settings - Fork 8
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
961 folder creation #1012
961 folder creation #1012
Conversation
def persisted? | ||
false | ||
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.
Style/EmptyLinesAroundBody: Extra empty line detected at body end.
c167f4f
to
c3ddd29
Compare
c3ddd29
to
8c3bff5
Compare
@@ -0,0 +1,35 @@ | |||
class RepositoryDirectoriesController < InheritedResources::Base |
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.
Style/FileName: Use snake_case for source file names.
8c3bff5
to
e9bd8ac
Compare
e9bd8ac
to
230a45d
Compare
protected | ||
|
||
def resource | ||
@repository_directory ||= RepositoryDirectory.new(params.merge(user: current_user)) |
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.
Metrics/LineLength: Line is too long. [87/80]
230a45d
to
ac98604
Compare
target_filename | ||
else | ||
temp_file.original_filename | ||
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.
Lint/EndAlignment: end
at 118, 4 is not aligned with if
at 114, 15
ac98604
to
6d51db2
Compare
Woof woof!
The 'Create subdirectory' button now adds a line to the file browser containing a form. When this form is sent, an XHR is sent. If the directory could be created, the form is replaced by a link. Otherwise an error message is shown. The user can show as many forms as he wants.
This was originally bringit code that showed information on the last commit of each file in the file browser. This was deactivated (but not entirely deleted) because of serious performance issues on big repositories.
5deb5f8
to
f95f01c
Compare
👍 |
fixes #961