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

allow 'Nextcloud' in the user agent string of Android #7468

Merged
merged 2 commits into from
Dec 12, 2017

Conversation

schiessle
Copy link
Member

backport of #7462

Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>
Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>
@schiessle schiessle added the 3. to review Waiting for reviews label Dec 12, 2017
@schiessle schiessle added this to the Nextcloud 12.0.5 milestone Dec 12, 2017
@MorrisJobke MorrisJobke added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Dec 12, 2017
@codecov
Copy link

codecov bot commented Dec 12, 2017

Codecov Report

Merging #7468 into stable12 will increase coverage by <.01%.
The diff coverage is n/a.

@@              Coverage Diff               @@
##             stable12    #7468      +/-   ##
==============================================
+ Coverage       53.92%   53.92%   +<.01%     
  Complexity      22703    22703              
==============================================
  Files            1384     1384              
  Lines           86959    86959              
  Branches         1330     1330              
==============================================
+ Hits            46891    46895       +4     
+ Misses          40068    40064       -4
Impacted Files Coverage Δ Complexity Δ
lib/private/Files/Cache/Propagator.php 94.93% <0%> (-1.27%) 16% <0%> (ø)
core/js/js.js 61.83% <0%> (+0.55%) 0% <0%> (ø) ⬇️

@MorrisJobke MorrisJobke merged commit 93797b2 into stable12 Dec 12, 2017
@MorrisJobke MorrisJobke deleted the update-user-agent-regex-stable12 branch December 12, 2017 14:44
@MorrisJobke MorrisJobke mentioned this pull request Jan 8, 2018
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants