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

Add certs to bedrock: bug 1149019 #3847

Merged
merged 1 commit into from Feb 11, 2016
Merged

Add certs to bedrock: bug 1149019 #3847

merged 1 commit into from Feb 11, 2016

Conversation

jgmize
Copy link
Contributor

@jgmize jgmize commented Feb 10, 2016

No description provided.

@jgmize
Copy link
Contributor Author

jgmize commented Feb 10, 2016

Fixes #3844

@jgmize
Copy link
Contributor Author

jgmize commented Feb 10, 2016

@pmclanahan r?

@pmclanahan
Copy link
Contributor

Looks good, but the files should be in media/ not static/.

@@ -12,11 +12,13 @@ class BedrockWhiteNoise(DjangoWhiteNoise):
def __init__(self, application):
super(BedrockWhiteNoise, self).__init__(application)

# TODO add tests for these in mcom-tests
# TODO add functional tests for these
Copy link
Contributor

Choose a reason for hiding this comment

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

flake8 doesn't like the trailing whitespace on the comment :( seems a bit picky, but okay.

@pmclanahan
Copy link
Contributor

Move the files and fix flake8 and we're good. Thanks!

@jgmize
Copy link
Contributor Author

jgmize commented Feb 11, 2016

Move the files and fix flake8 and we're good. Thanks!

Thanks @pmclanahan, good catch on the dir. I've fixed that and removed the trailing space in 6edb8a8

pmclanahan added a commit that referenced this pull request Feb 11, 2016
Add certs to bedrock: bug 1149019
@pmclanahan pmclanahan merged commit b50f563 into mozilla:master Feb 11, 2016
@jgmize jgmize deleted the certs branch February 11, 2016 04: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.

None yet

2 participants