lookit / lookit-api

Codebase for Lookit v2 and Experimenter v2. Includes an API. Docs: http://lookit.readthedocs.io/
https://lookit.mit.edu/
MIT License
10 stars 18 forks source link

Fix data availability with survey-consent #1330

Closed becky-gilbert closed 10 months ago

becky-gilbert commented 10 months ago

Fixes #1317

This PR flags any responses that use the survey-consent frame on the consent manager page. Specifically:

I made a few other UI changes while working on the consent manager page:

Screenshots:

Initial page with no response selected:

consent-no-selection

Response selected that contains a survey-consent form and no video found:

consent-survey-consent-update

Response selected with no video found:

consent-no-video-update

Response with a comment (I removed the 'N/A' text when there's no comment, but actual comments still show up as italicized text, as before):

consent-comment-update

More screenshots and a video here: https://github.com/lookit/lookit-api/issues/1317#issuecomment-1853349186

sonarcloud[bot] commented 10 months ago

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud