Only load for apps/files and apps/files_sharing #635

Merged
merged 1 commit into from Mar 30, 2016

Projects

None yet

6 participants

@rullzer
Contributor
rullzer commented Mar 30, 2016

Second try for #619

@PVince81 @oparoz please have a look.

@rullzer rullzer Only load for apps/files and apps/files_sharing
Round two for #619
287da95
@rullzer rullzer added the bug label Mar 30, 2016
@mention-bot

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

@rullzer rullzer added this to the 9.1-current milestone Mar 30, 2016
@PVince81
Contributor

👍 looks good

@oparoz
Collaborator
oparoz commented Mar 30, 2016

Test plan is missing, so reviewers, make sure you test this case: #619 (comment)

@SergioBertolinSG
Member

👍 works (also #635 (comment))

@oparoz
Collaborator
oparoz commented Mar 30, 2016

Thank you! I'll check later today.

@PVince81
Contributor

I'll leave it up to you to merge it @oparoz (and the backport), hopefully before the end of the week so we can have it in the next RC.

@oparoz
Collaborator
oparoz commented Mar 30, 2016

I will, thank @PVince81

@oparoz oparoz merged commit f440160 into master Mar 30, 2016

2 checks passed

Scrutinizer No new issues
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@oparoz oparoz deleted the 619_round2 branch Mar 30, 2016
@oparoz oparoz removed the in progress label Mar 30, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment