Skip to content

Conversation

@susnux
Copy link
Contributor

@susnux susnux commented Aug 7, 2024

When splitting the path it is /{ID} to the splitted array is ['', '{ID}'] -> we need to use the last part otherwise we only get 404.

@susnux susnux added this to the Nextcloud 30 milestone Aug 7, 2024
@susnux susnux requested review from Pytal, nfebe, skjnldsv and sorbaugh August 7, 2024 23:12
@susnux
Copy link
Contributor Author

susnux commented Aug 7, 2024

/backport to stable29

@susnux
Copy link
Contributor Author

susnux commented Aug 7, 2024

/backport to stable28

@susnux
Copy link
Contributor Author

susnux commented Aug 7, 2024

/compile

@susnux susnux mentioned this pull request Aug 7, 2024
Signed-off-by: nextcloud-command <[email protected]>
@skjnldsv
Copy link
Member

skjnldsv commented Aug 8, 2024

We need tests for this 🙈

@AndyScherzinger
Copy link
Member

Merged to have it in Beta5 but created a follow-up at #47127 to have an issue for implementing tests

@susnux
Copy link
Contributor Author

susnux commented Aug 8, 2024

We need tests for this 🙈

I am already writing some, but this was more urgent 😅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants