Inline JS is not great because there's no clear definition of the interface between the view and the javascript and it makes it more difficult to take advantage of a lot of the JavaScript tooling that's out there.

I would do this as something like

```javascript
var EDIT_COMMCARE_USER_VARS = {
    requestPath: '{{ request.path }}',
    commTrack: {{ commtrack|BOOL }},
    userId: '{{ couch_user.user_id }}'
    userDataMessage: '{% trans 'User Data' %}',
    passwordChangedMessage: '{% trans "Password Changed Successfully" %}',
    isCurrentlyLoggedInUser: '{{ is_currently_logged_in_user }}'
};
```

and then change the references in the JS to EDIT_COMMCARE_USER_VARS.foo instead of having template variable includes.

That way the next person can easily see all of the things that the JavaScript on the page depends on instead of having to scan the template JS for variables.