-
Notifications
You must be signed in to change notification settings - Fork 186
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
Use nextclouds temp dir #493
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks fine to me but I really have no idea.
I did test it on my local machine and it seems to work fine: |
If @kesselb doesn't respond I'll approve it, but I think the person who pointed out the problem would be uniquely qualified to review the fix. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me
Still looks fine to me |
…reen * origin/master: (32 commits) prevent menu actions from collapsing sidebar (nextcloud#504) add "mark read" floating button mobile. Fixes nextcloud#469 (nextcloud#505) [tx-robot] updated from transifex [tx-robot] updated from transifex update js dependencies (nextcloud#500) [tx-robot] updated from transifex removed warning about non utf8mb4 charset for mysql/mariadb - nextcloud#496 (nextcloud#497) [tx-robot] updated from transifex [tx-robot] updated from transifex Update feed-io: adds UT timezone (nextcloud#495) [tx-robot] updated from transifex [tx-robot] updated from transifex Release 13.1.6 (nextcloud#490) Use nextclouds temp dir (nextcloud#493) [tx-robot] updated from transifex [tx-robot] updated from transifex UI: Make BasicAuth Credentials less important in subscribtion form Set tmp dir as cache for favicon embed codecov status remove coverage from integration run, upload to codecov ... Signed-off-by: nachoparker <nacho@ownyourbits.com>
#485 (comment)