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

Fix opening files #1651

Merged
merged 4 commits into from
Oct 27, 2017
Merged

Fix opening files #1651

merged 4 commits into from
Oct 27, 2017

Conversation

mario
Copy link
Contributor

@mario mario commented Oct 10, 2017

Some apps are stupid and won't allow us to open files in write mode. This fixes it.

I think this is important for 2.0, but you decide...

resolves #751

Signed-off-by: Mario Danic mario@lovelyhq.com

@kangaroo72
Copy link

Is it available in beta?

@mario
Copy link
Contributor Author

mario commented Oct 10, 2017

No, but it works. :P Hopefully we'll have it in 2.0.

@kangaroo72
Copy link

I don't see it on my mobile. I'm not at a Computer... Do you have a way to do or a working apk?

@AndyScherzinger
Copy link
Member

AndyScherzinger commented Oct 10, 2017

👍 test on my Android 7.1.1 device is fine, can't say anything about older Android versions.

Approved with PullApprove

@kangaroo72
Copy link

I can report several devices, but I'm unable to code ... If someone can integrate this in the rc-version from store or delivering an apk, I will check. Thanks to all - extreme good work!!!!!!

@AndyScherzinger
Copy link
Member

@mario as for 2.0.0 I'd prefer if we just release 2.0.0 and put this into a 2.0.1 via a bugfix branch 2.0.x to not having to do another RC and not postpone 2.0.0 (due to golden master). Just my opinion though!

@mario
Copy link
Contributor Author

mario commented Oct 11, 2017 via email

@tobiasKaminsky
Copy link
Member

Yes, same opinion as Andy.
It seems not to be a regression, but "just" a regular bugfix. So either 2.0.1 which we then can do very shortly after 2.0 or 2.1 which I would like to start RC by end of October with all the Hacktoberfest PRs in it.

@mario
Copy link
Contributor Author

mario commented Oct 11, 2017 via email

@AndyScherzinger
Copy link
Member

Well, for 2.1 I'd also be fine with just Hacktober + open PRs

@AndyScherzinger
Copy link
Member

I'd also be fine with having two releases within 2 month or so, like putting the rest into 2.2 but getting back to a release schedule of 4-6 weeks for bugfix/feature releases maybe 2 month tops sometimes.

@mario
Copy link
Contributor Author

mario commented Oct 11, 2017

KO, whatever you two decide, but I need this FAST. Same with other PR I'm gonna open.

@kangaroo72
Copy link

I don't want to hurry. But I would also be very thankful if someone can provide me a apk-file for testing. I can check it on several Android-Systems-/Devices and will report results. Thanks, Team

@mario mario mentioned this pull request Oct 11, 2017
@mario
Copy link
Contributor Author

mario commented Oct 11, 2017

@kangaroo72 it's gonna be in the next daily build on f-droid.

@kangaroo72
Copy link

@mario cool - I'll check - thank you!!!!

@mario mario mentioned this pull request Oct 11, 2017
@kangaroo72
Copy link

@mario - Wenn the next daily build on f-droid arrives, is it the "nextcloud-dev" app?

@mario
Copy link
Contributor Author

mario commented Oct 13, 2017 via email

@mario
Copy link
Contributor Author

mario commented Oct 15, 2017

@kangaroo72 news? :)

@kangaroo72
Copy link

eeeh - not yet? F-Droid shows Version 20170930 ...

@mario
Copy link
Contributor Author

mario commented Oct 17, 2017

Well, we won't have 2.0.1 unless something major happens - 2.1 is the next release! :)

@mario
Copy link
Contributor Author

mario commented Oct 17, 2017

(but F-droid Dev should have built it, but I'm no F-droid expert)

@kangaroo72
Copy link

Isn't it possible to submit your apk?

@AndyScherzinger
Copy link
Member

No, f-droid builds the app from source themselves...

@AndyScherzinger AndyScherzinger modified the milestones: Nextcloud App 2.0.1, Nextcloud App 2.1.0 Oct 19, 2017
@AndyScherzinger AndyScherzinger force-pushed the fix-word-opening branch 2 times, most recently from 7810aec to c738f9b Compare October 20, 2017 18:46
Copy link
Contributor

@xXSTrikeXx xXSTrikeXx left a comment

Choose a reason for hiding this comment

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

sweet

@AndyScherzinger
Copy link
Member

pinging @tobiasKaminsky for review, see tested successfully on my OPT3 - #1651 (comment)

@tobiasKaminsky
Copy link
Member

tobiasKaminsky commented Oct 27, 2017

👍

I'll rebase and merge once #1559 is done.

mario and others added 4 commits October 27, 2017 08:04
Signed-off-by: Mario Danic <mario@lovelyhq.com>
Signed-off-by: Mario Danic <mario@lovelyhq.com>
Signed-off-by: Mario Danic <mario@lovelyhq.com>
@fhempeter
Copy link

Downloaded the latest developer Version apk file today, issue is still there. Can you please advise where to download an apk that includes this fix ? Thank you !

@tobiasKaminsky
Copy link
Member

This will be available as soon as #1775 is merged and run.

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.

Edit on the fly possible (*.xlsx, *.docx, *.txt ...)?
7 participants