Skip to content

Conversation

@AndyScherzinger
Copy link
Member

Backport of #2700 (create an email share)

Backport tested sucessfully

cc @tobiasKaminsky @mario for review

tobiasKaminsky and others added 8 commits June 18, 2018 18:45
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>

# Conflicts:
#	build.gradle
# Conflicts:
#	src/main/java/com/owncloud/android/ui/fragment/util/SharingMenuHelper.java
@AndyScherzinger AndyScherzinger added this to the Nextcloud App 3.2.1 milestone Jun 18, 2018
@AndyScherzinger AndyScherzinger changed the base branch from master to stable-3.2.x June 18, 2018 16:55
@tobiasKaminsky tobiasKaminsky mentioned this pull request Jun 18, 2018
24 tasks
@AndyScherzinger
Copy link
Member Author

@tobiasKaminsky Tried close/reopen but for some reason DroneCI doesn't detect this PR and thus doesn't trigger a build :(

@tobiasKaminsky
Copy link
Member

I re-checked every settings and they seem ok, @MorrisJobke?

@MorrisJobke
Copy link
Member

The CI job itself ran fine: https://drone.nextcloud.com/nextcloud/android/5122

Let me retrigger it on CI - the status will update once it's done.

@tobiasKaminsky tobiasKaminsky force-pushed the backportMailShareCreation branch from 064c294 to 3b7ca9e Compare June 19, 2018 08:00
@codecov
Copy link

codecov bot commented Jun 19, 2018

Codecov Report

Merging #2724 into stable-3.2.x will increase coverage by 0.01%.
The diff coverage is 9.09%.

@@               Coverage Diff               @@
##           stable-3.2.x   #2724      +/-   ##
===============================================
+ Coverage          6.53%   6.54%   +0.01%     
===============================================
  Files               283     283              
  Lines             28861   28861              
  Branches           4199    4202       +3     
===============================================
+ Hits               1885    1888       +3     
+ Misses            26691   26689       -2     
+ Partials            285     284       -1
Impacted Files Coverage Δ
...in/java/com/owncloud/android/utils/ThemeUtils.java 0% <0%> (ø) ⬆️
...oid/operations/CreateShareWithShareeOperation.java 0% <0%> (ø) ⬆️
...cloud/android/ui/helpers/FileOperationsHelper.java 0% <0%> (ø) ⬆️
...ndroid/providers/UsersAndGroupsSearchProvider.java 17.02% <25%> (+0.35%) ⬆️
.../third_parties/daveKoeller/AlphanumComparator.java 82.75% <0%> (+2.29%) ⬆️

@tobiasKaminsky
Copy link
Member

I removed "backport-request" as this label is for those PRs, that need to be backported.
For this one it is totally fine to add it to the milestone and to the release overview, as you already did :-)

@AndyScherzinger AndyScherzinger requested review from mario and tobiasKaminsky and removed request for mario June 20, 2018 10:03
@mario
Copy link
Contributor

mario commented Jun 21, 2018

👍

Approved with PullApprove

@tobiasKaminsky
Copy link
Member

tobiasKaminsky commented Jun 22, 2018

👍

Approved with PullApprove

@AndyScherzinger AndyScherzinger merged commit bb5cf9d into stable-3.2.x Jun 23, 2018
@AndyScherzinger AndyScherzinger deleted the backportMailShareCreation branch June 23, 2018 07:37
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.

5 participants