[TASK-82] Feature: Project-level control of anonymous submission #4719
Conversation
…ned to anonymous user
Conflicts: jsapp/js/account/accountSettingsRoute.tsx
noliveleger
changed the title
Add submissions permission assignment to anonymous user
noliveleger
changed the base branch from
feature/per-project-anonymous-access
to
beta
noliveleger
changed the title
Add submissions permission assignment to anonymous userAdd submissions permission assignment to anonymous user
noliveleger
changed the title
Add submissions permission assignment to anonymous userAdd submissions permission assignment to anonymous user
|
The tooltip in the Anonymous Submission component is causing sizing issues in the forms it has placed in. It seems to be making the form containers horizontally scrollable. For example, in the modal sharing form: And in the form landing component: Cutoff tooltip: |
Adjust anon submission tooltip layout issue
that it is for Enketo
| except (User.extra_details.RelatedObjectDoesNotExist, KeyError): | ||
| require_auth = False | ||
| require_auth = obj.asset.deployment.xform.require_auth | ||
| except (DeploymentNotFound, AttributeError): |
There was a problem hiding this comment.
Kind of a weird situation, but I guess AttributeError is needed because it's possible for the xform property to return None?
| 'kpi.deployment_backends.backends.MockDeploymentBackend.xform', | ||
| MagicMock(), | ||
| ) as xf_mock: | ||
| type(xf_mock).require_auth = PropertyMock(return_value=False) |
There was a problem hiding this comment.
This is wild. If you explain it to me, I might learn something
There was a problem hiding this comment.
Unfortunately, you won't learn anything today. It's a wrong try/error mock of a non-existing property (i.e.: MockDeploymentBackend.xform) but I could not make it work so I added a dummy property xform to MockDeploymentBackend.
Long story short: this can be simplified with xf_mock.require_auth = False
noliveleger
force-pushed
the
require-auth-at-project-level
branch
2 times, most recently
from
31c5143 to
f496808
Compare
noliveleger
force-pushed
the
require-auth-at-project-level
branch
from
f496808 to
e4d6ad3
Compare
Description
Account setting
Require auth to see forms and datais not used anymore and OpenRosa server URL is updated based on anonymous userAdd submissionspermission.Add submissionsdo NOT implyView formanymore.Additional info
A new environment variable has been introduced
ENKETO_REDIS_MAIN_URLand must be set.require_authfield has been removed from reports (Admin and Project views)Notes
OpenRosa server url is modified directly in redis for Enketo to comply Kobocat (new) requirements. i.e.: Projects with required auth are listed at https://kobocat/ and projects which allow anonymous submissions are listed at https://kobocat/
Related issues
Needs kobotoolbox/kobo-install#236
Blocked by
kobotoolbox/kobocat#904