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.

Sign up for GitHub

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

Jump to bottom

fix: Properly take show_hidden user setting into account for file listing #42006

Merged
merged 2 commits into from
Dec 4, 2023
Merged

fix: Properly take show_hidden user setting into account for file listing #42006

merged 2 commits into from
Dec 4, 2023

Conversation

juliushaertl
Copy link
Member

@juliushaertl juliushaertl commented Dec 4, 2023 •
edited
Loading

Fix #41448

Checklist

@juliushaertl juliushaertl added bug 3. to review Waiting for reviews labels Dec 4, 2023
@juliushaertl juliushaertl requested a review from a team December 4, 2023 11:36
@juliushaertl juliushaertl added this to the Nextcloud 29 milestone Dec 4, 2023
@juliushaertl
Copy link
Member Author

/backport to stable28

ChristophWurst
ChristophWurst approved these changes Dec 4, 2023
View reviewed changes
Copy link
Member

@ChristophWurst ChristophWurst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you

@juliushaertl
fix: Properly take show_hidden user setting into account for file lis…
d9ae4b6 
…ting

Signed-off-by: Julius Härtl <jus@bitgrid.net>
@juliushaertl
Copy link
Member Author

/compile

@nextcloud-command
chore(assets): Recompile assets
ca684c9 
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@juliushaertl juliushaertl merged commit ca60df9 into master Dec 4, 2023
38 of 41 checks passed
@juliushaertl juliushaertl deleted the bugfix/41448 branch December 4, 2023 21:09
@juliushaertl
Copy link
Member Author

Well seems the failure was not as unrelated as I thought initially, fix in #42021

This was referenced Dec 4, 2023
Merged
Merged
@juliushaertl
Copy link
Member Author

backport in #42023

@blizzz blizzz mentioned this pull request Mar 5, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AndyScherzinger AndyScherzinger AndyScherzinger approved these changes

@ChristophWurst ChristophWurst ChristophWurst approved these changes

@artonge artonge Awaiting requested review from artonge

@susnux susnux Awaiting requested review from susnux

@skjnldsv skjnldsv Awaiting requested review from skjnldsv skjnldsv is a code owner

Assignees
No one assigned
Labels
3. to review Waiting for reviews bug
Projects
None yet
Milestone
Nextcloud 29
Development

Successfully merging this pull request may close these issues.

[Bug]: Dot files (hidden files) are now always shown
4 participants
@juliushaertl @AndyScherzinger @ChristophWurst @nextcloud-command