Skip to content
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

Write files in subdirectories to splat path #1513

Merged
merged 1 commit into from Apr 28, 2017
Merged

Conversation

@PtrTeixeira
Copy link
Contributor

PtrTeixeira commented Apr 20, 2017

When the page was loaded the first time, the call to get the files to
the directory wasn't being called with the directory splat. It was
therefore always fetching the files from the top-level directory. When
linked to the top level directory, this wasn't notable, but links into
the subdirectory of the file browser would seem as if the directory was
empty until going back to the root directory.

When the page was loaded the first time, the call to get the files to
the directory wasn't being called with the directory splat. It was
therefore always fetching the files from the top-level directory. When
linked to the top level directory, this wasn't notable, but links into
the subdirectory of the file browser would seem as if the directory was
empty until going back to the root directory.
@ssalinas
Copy link
Member

ssalinas commented Apr 21, 2017

Gotta love when one missing variable can ruin the day... LGTM

@PtrTeixeira PtrTeixeira added the hs_qa label Apr 25, 2017
@ssalinas ssalinas added the hs_stable label Apr 26, 2017
@ssalinas ssalinas modified the milestone: 0.15.0 Apr 28, 2017
@ssalinas ssalinas merged commit 163ddbb into master Apr 28, 2017
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@ssalinas ssalinas deleted the load-subdirectory-links branch Apr 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.