Skip to content

Fix AxiosError when deleting photo in viewer#1560

Merged
skjnldsv merged 1 commit intomasterfrom
umgfoin_PR_issue_1591
Feb 27, 2023
Merged

Fix AxiosError when deleting photo in viewer#1560
skjnldsv merged 1 commit intomasterfrom
umgfoin_PR_issue_1591

Conversation

@umgfoin
Copy link
Contributor

@umgfoin umgfoin commented Feb 27, 2023

const url = this.source ?? this.root + this.currentFile.filename
constructs illegal url - use this.currentFile.davPath instead.

Fixes` issue #1591: AxiosError: Request failed with status code 404 when deleting photo

@umgfoin umgfoin added the bug Something isn't working label Feb 27, 2023
@umgfoin umgfoin self-assigned this Feb 27, 2023
@umgfoin umgfoin requested a review from skjnldsv February 27, 2023 10:49
@skjnldsv
Copy link
Member

/backport to stable25

@backportbot-nextcloud backportbot-nextcloud bot added the backport-request Pending backport by the backport-bot label Feb 27, 2023
@skjnldsv skjnldsv added 4. to release Ready to be released and/or waiting for tests to finish and removed backport-request Pending backport by the backport-bot labels Feb 27, 2023
@skjnldsv
Copy link
Member

/rebase

@skjnldsv
Copy link
Member

/compile amend /

Fixes issue #1591: Axios error when deleting photos in viewer

Signed-off-by: umgfoin <umgfoin@users.noreply.github.com>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@skjnldsv skjnldsv enabled auto-merge February 27, 2023 11:07
@skjnldsv skjnldsv merged commit c926f18 into master Feb 27, 2023
@skjnldsv skjnldsv deleted the umgfoin_PR_issue_1591 branch February 27, 2023 11:08
@backportbot-nextcloud backportbot-nextcloud bot added the backport-request Pending backport by the backport-bot label Feb 27, 2023
@backportbot-nextcloud backportbot-nextcloud bot removed the backport-request Pending backport by the backport-bot label Feb 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to release Ready to be released and/or waiting for tests to finish bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants