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(files): fix new folder encoding #40948

Merged
merged 1 commit into from Oct 17, 2023
Merged

fix(files): fix new folder encoding #40948

merged 1 commit into from Oct 17, 2023

Conversation

skjnldsv
Copy link
Member

@skjnldsv skjnldsv commented Oct 17, 2023 •
edited

  • Fix creating new folders not refreshing the view because of the encoded path
  • Also fix new folders showing in the list as encoded
    Peek 17-10-2023 15-01

@skjnldsv skjnldsv added this to the Nextcloud 28 milestone Oct 17, 2023
@skjnldsv skjnldsv requested a review from a team October 17, 2023 12:59
@skjnldsv skjnldsv added this to In progress in Files to vue via automation Oct 17, 2023
@skjnldsv skjnldsv self-assigned this Oct 17, 2023
@skjnldsv skjnldsv requested review from Fenn-CS, szaimen, sorbaugh and susnux and removed request for a team October 17, 2023 12:59
@skjnldsv skjnldsv moved this from In progress to Potential regressions in Files to vue Oct 17, 2023
@susnux

This comment was marked as resolved.

Sign in to view
@skjnldsv
fix(files): fix new folder encoding
9f157d7 
Signed-off-by: John Molakvoæ <skjnldsv@protonmail.com>
@skjnldsv skjnldsv added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Oct 17, 2023
@skjnldsv skjnldsv merged commit 592eac0 into master Oct 17, 2023
38 of 40 checks passed
Files to vue automation moved this from Potential regressions to Done Oct 17, 2023
@skjnldsv skjnldsv deleted the fix/new-folder branch October 17, 2023 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@susnux susnux susnux approved these changes

@Altahrim Altahrim Altahrim approved these changes

@Fenn-CS Fenn-CS Awaiting requested review from Fenn-CS Fenn-CS was automatically assigned from nextcloud/server-frontend

@szaimen szaimen Awaiting requested review from szaimen szaimen was automatically assigned from nextcloud/server-frontend

@sorbaugh sorbaugh Awaiting requested review from sorbaugh sorbaugh was automatically assigned from nextcloud/server-frontend

Assignees

@skjnldsv skjnldsv

Labels
4. to release Ready to be released and/or waiting for tests to finish bug feature: files
Projects
Files to vue
  
Done
Milestone
Nextcloud 28
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@skjnldsv @susnux @Altahrim