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

updates for 2.0 release #1627

Merged
merged 6 commits into from
May 4, 2016
Merged

updates for 2.0 release #1627

merged 6 commits into from
May 4, 2016

Conversation

carlaschroder
Copy link

Allrighty friends, this release looks gorgeous.

I did not see the upload views shown in #1551 I tried uploading multiple files, which works fine, but the uploads indicator is just a little message on the top left.

@rperezb @davivel @AndyScherzinger

@mention-bot
Copy link

By analyzing the blame information on this pull request, we identified @danimo to be a potential reviewer

@AndyScherzinger
Copy link
Contributor

Hi @carlaschroder the issue you are refering to is #1493

  • what is the version displayed in the settings screen at the bottom?
  • do you have a drawer menu item (via top left) that says "Uploads"?

@davivel
Copy link
Contributor

davivel commented Apr 22, 2016

@carlaschroder , the uploads view is not show automatically. You need to open the side drawer sliding from the left border of the list of files, or clicking on the burger menu. Then click on "Uploads"

@davivel
Copy link
Contributor

davivel commented Apr 22, 2016

@carlaschroder , for next release we'll ping you before doing the release, I promise; we need to take the documentation more seriously.

cc @rperezb

@rperezb
Copy link

rperezb commented Apr 22, 2016

@davivel we take documentation very seriously 👼 I have open several issues on the doc repo

@rperezb
Copy link

rperezb commented Apr 22, 2016

@carlaschroder on your screenshot user_manual/images/android-5.png you may notice that there is a new section: Uploads this is it!

@carlaschroder
Copy link
Author

for next release we'll ping you before doing the release

@rperezb has kept me well-informed. That is why the updates are out now instead of weeks from now :)

@carlaschroder
Copy link
Author

dangit, this out of sync with the master branch already. Stop doing that :)

@carlaschroder
Copy link
Author

ok then, a new commit describing the update status indicator.
@rperezb @davivel @AndyScherzinger

shortcut to your files, the **Uploads** button, and the **Settings** button.

The Uploads page displays the status of files currently uploading, a list of
your recently-uploaded files, and a Retry option for any failed uploads.
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Plus I'd add that it's possible to fix failed uploads, the meaning of this is that:

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

gottit, thanks!

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mmmm, I could be wrong, but I don't remember we implemented the recovery option for "Privileges error".

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you find out? It shouldn't take weeks to approve documentation :)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can confirm, Carla. An upload to a folder where the user cannot write ends with a failed upload with "Permission error". If the users taps on it in the uploads view, the upload is directly retried, finishing again in "Permission error". Unless something is done outside the app to grant the permission, the upload will always fail

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome sauce, thanks. Corrections committed.

@carlaschroder
Copy link
Author

See how you like it now :)

@carlaschroder
Copy link
Author

Is this good to merge yet? It's not getting any fresher sitting here :)
@rperezb @davivel @AndyScherzinger

# The full version, including alpha/beta/rc tags.
release = '1.9.1'
release = '2.0'

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For consistency throughout the project format of version should always be three-fold x.y.z, so 2.0.0 here.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed :)

@carlaschroder
Copy link
Author

sigh. I'm done-- you nice people fix it and merge it.

@davivel
Copy link
Contributor

davivel commented May 3, 2016

@carlaschroder , sorry for the pain :(

I will give it a final look tomorrow and go all the way 'til merging. Promised.

@carlaschroder
Copy link
Author

Thanks @davivel , and thanks @LukeOwncloud.

@AndyScherzinger
Copy link
Contributor

@davivel Review comments resolved and branch rebased to master. Feel free to check and hit the merge button :)

@carlaschroder carlaschroder merged commit f9716fd into master May 4, 2016
@carlaschroder
Copy link
Author

I feel the power! Thanks everyone.

@carlaschroder carlaschroder deleted the 2-0-manual branch May 4, 2016 14:23
@davivel
Copy link
Contributor

davivel commented May 4, 2016

👍 Thanks @AndyScherzinger

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

Successfully merging this pull request may close these issues.

6 participants