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.

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

1.4.2 Video upload folder destination incorrect #770

Closed
AndyScherzinger opened this issue Mar 23, 2017 · 7 comments
Closed

1.4.2 Video upload folder destination incorrect #770

AndyScherzinger opened this issue Mar 23, 2017 · 7 comments

Comments

@AndyScherzinger
Copy link
Member

AndyScherzinger commented Mar 23, 2017

see https://help.nextcloud.com/t/1-4-2-video-upload-folder-destination-incorrect/10432

There is an option to have it upload to <folder>/<year>/<month> (folder being user selected) this is broken in 1.4.2

In this case my destination folder for my videos is "video".

In 1.4.2. it creates a folder called <folder>+<year>/<Month> i.e. video2017/03 when the path should have been video/2017/03


I haven't tested this personally but know that we fixed this issue pre-1.4.2 final for images, but am not sure if this is then also automatically fixed for video uploads (but would think so).

@AndyScherzinger
Copy link
Member Author

probably fixed by #774 - waiting for the user feedback on the Android version in use.

@AndyScherzinger AndyScherzinger added this to the Nextcloud App 1.5.0 milestone Mar 24, 2017
@DaveBurkhardt
Copy link

DaveBurkhardt commented Mar 24, 2017

screenshot_2017-03-24-16-27-38

screenshot_2017-03-24-16-27-21

Validation images.

@mario
Copy link
Contributor

mario commented Mar 24, 2017

@DaveBurkhardt thanks. weather doesn't help though :D

@mario
Copy link
Contributor

mario commented Mar 24, 2017

This will either be fixed in the next release or we'll replace instant upload with auto upload which doesn't have separation between images and videos. Thanks for reporting.

@DaveBurkhardt
Copy link

@mario haha, sorry! Fixed above

@mario
Copy link
Contributor

mario commented Mar 24, 2017

@DaveBurkhardt all good! :)

@AndyScherzinger
Copy link
Member Author

PR has been merged, to be shipped with 1.5.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants