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 invalid theming URLs #4707

Merged
merged 1 commit into from
May 5, 2017
Merged

Fix invalid theming URLs #4707

merged 1 commit into from
May 5, 2017

Conversation

MorrisJobke
Copy link
Member

@mention-bot
Copy link

@MorrisJobke, thanks for your PR! By analyzing the history of the files in this pull request, we identified @juliushaertl, @LukasReschke and @schiessle to be potential reviewers.

* logo and background image had the cache buster applied twice:
  background-image: url(http://192.168.99.100/server/core/img/logo.svg?v=0?v=0);

Signed-off-by: Morris Jobke <hey@morrisjobke.de>
@codecov
Copy link

codecov bot commented May 5, 2017

Codecov Report

Merging #4707 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #4707      +/-   ##
============================================
+ Coverage     54.27%   54.28%   +<.01%     
  Complexity    22099    22099              
============================================
  Files          1361     1361              
  Lines         84708    84709       +1     
  Branches       1322     1322              
============================================
+ Hits          45978    45984       +6     
+ Misses        38730    38725       -5
Impacted Files Coverage Δ Complexity Δ
apps/theming/lib/ThemingDefaults.php 87.5% <100%> (+0.12%) 33 <0> (ø) ⬇️
lib/private/Server.php 93.37% <0%> (-0.15%) 120% <0%> (ø)
core/js/js.js 61.78% <0%> (+0.56%) 0% <0%> (ø) ⬇️
lib/private/Security/CertificateManager.php 93.81% <0%> (+1.03%) 38% <0%> (ø) ⬇️

Copy link
Member

@juliushaertl juliushaertl left a comment

Choose a reason for hiding this comment

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

👍

@rullzer rullzer merged commit 15f43b2 into master May 5, 2017
@rullzer rullzer deleted the fix-invalid-theming-urls branch May 5, 2017 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews regression
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants