Add django.core.context_processors.request

RESOLVED WONTFIX

Status

Webtools
Elmo
P5
normal
RESOLVED WONTFIX
8 years ago
7 years ago

People

(Reporter: stas, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [good first bug])

Attachments

(1 attachment)

(Reporter)

Description

8 years ago
Created attachment 436187 [details] [diff] [review]
Add TEMPLATE_CONTEXT_PROCESSORS to l10n_site.settings

I'd like to be able to use the request object inside templates (e.g. {{ request.get_full_path }})

This requires a change to l10n_site.settings: adding the django.core.context_processors.request context processor. 

There are 4 processors that are enabled by default. We'll need to list them explicitly in TEMPLATE_CONTEXT_PROCESSORS.
Attachment #436187 - Flags: review?(l10n)
Are you going to provide custom context processors or do you just want to use the default ones?
(Reporter)

Comment 2

8 years ago
No custom, I just need django.core.context_processors.request which is not enabled by default.
(Reporter)

Comment 3

8 years ago
In fact, let's put this on hold for now. I might not need it after all.

Updated

8 years ago
Blocks: 562215

Updated

8 years ago
No longer blocks: 562215

Comment 4

8 years ago
Comment on attachment 436187 [details] [diff] [review]
Add TEMPLATE_CONTEXT_PROCESSORS to l10n_site.settings

r- based on the changed impact of this patch.

This should modify the pushes views, and the base.html template, to remove the need to pass in request explicitly when specifying login_form_needs_reload.
Attachment #436187 - Flags: review?(l10n) → review-

Updated

8 years ago
Priority: -- → P5
Whiteboard: [good first bug]
(Reporter)

Updated

7 years ago
Assignee: nobody → stas

Updated

7 years ago
Assignee: stas → nobody
Component: Infrastructure → Elmo
Product: Mozilla Localizations → Webtools
QA Contact: infrastructure → elmo
Summary: [dashboard] Add django.core.context_processors.request → Add django.core.context_processors.request
Version: unspecified → 1.0
(Reporter)

Comment 5

7 years ago
We don't need this anymore.
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.