langcog / web-cdi

7 stars 5 forks source link

updating study reverts to old settings for demographics and sharing #472

Closed vmarchman closed 11 months ago

vmarchman commented 1 year ago

@HenryMehta We thought this was user error, but indeed, when someone updates a study that has previously opted out of sharing/demographics, the settings revert to the default. Not terribly urgent, but could be annoying for some users.

vmarchman commented 1 year ago

Maybe the solution is to simply add those choices to the update study dialog

HenryMehta commented 1 year ago

@vmarchman deploying update to dev now

vmarchman commented 1 year ago

Hi @Henry @.***> I'm confused by the options now. In the default version of the New Study dialog, in the Demographic options, I can click on a new demographic opt out option, even if I haven't selected opted out of sharing. but, then when I click on the opt out of sharing, I get the standard opt out option.

[image: image.png]

On Sun, Sep 3, 2023 at 10:01 PM Henry Mehta @.***> wrote:

@vmarchman https://github.com/vmarchman deploying update to dev now

— Reply to this email directly, view it on GitHub https://github.com/langcog/web-cdi/issues/472#issuecomment-1704613348, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB2TUTBWZLTEP2LDGHMDYX3XYVOBJANCNFSM6AAAAAA4HWRSWU . You are receiving this because you were mentioned.Message ID: @.***>

HenryMehta commented 1 year ago

@vmarchman because you didn't respond on GitHub I cannot see the image but I think I know your issue.

We have a new checkbox within Demogrpahics which will be used by the UCSD team, and anyone else who wants to, by which we collect no demographic data at all. This is not about sharing information. This is used when the demographic data is being collected within RedCAP as an example.

If you do select this option, you will see in the links you have to share that age, offset and sex need to be included within the link to WebCDI.

vmarchman commented 1 year ago

@Henry @.***> Sorry - but, we don't want folks to opt out of demographics unless they have opted out of sharing. If folks are collecting demographic information in RedCap we don't have access to that information within WebCDI, so it is not useful to us.

Can't we put that information in the dialog about setting up RedCap?

I don't want it to be a general option. Right now, it will just cause confusion.

On Mon, Sep 4, 2023 at 9:39 AM Henry Mehta @.***> wrote:

@vmarchman https://github.com/vmarchman because you didn't respond on GitHub I cannot see the image but I think I know your issue.

We have a new checkbox within Demogrpahics which will be used by the UCSD team, and anyone else who wants to, by which we collect no demographic data at all. This is not about sharing information. This is used when the demographic data is being collected within RedCAP as an example.

If you do select this option, you will see in the links you have to share that age, offset and sex need to be included within the link to WebCDI.

— Reply to this email directly, view it on GitHub https://github.com/langcog/web-cdi/issues/472#issuecomment-1705524360, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB2TUTDB66UYRE66LRCV4NDXYX72XANCNFSM6AAAAAA4HWRSWU . You are receiving this because you were mentioned.Message ID: @.***>

HenryMehta commented 1 year ago

@vmarchman ok, I'll amend tomorrow morning so you can only opt out of demographics if opted out of sharing

HenryMehta commented 1 year ago

@vmarchman Amendment deployed

vmarchman commented 1 year ago

Hi @Henry @.***> Thank you.

However, I'm still not sure about the 2 options currently in Demographic options. I know the one is for RedCap users, etc., but I fear folks will choose it and get confused.

Options?

On Mon, Sep 4, 2023 at 11:20 PM Henry Mehta @.***> wrote:

@vmarchman https://github.com/vmarchman Amendment deployed

— Reply to this email directly, view it on GitHub https://github.com/langcog/web-cdi/issues/472#issuecomment-1706009331, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB2TUTHQ63OXBBNDD2MVGUDXY3AC5ANCNFSM6AAAAAA4HWRSWU . You are receiving this because you were mentioned.Message ID: @.***>

HenryMehta commented 1 year ago

@vmarchman I have updated the text/explanation around the two checkboxes. Does this help

vmarchman commented 1 year ago

Yes @Henry @.***> Thank you!

(1) One of the options appears grey out, and one appears only if you choose opt out. Can both only appear when opt out?

(2) Can you change the order of the two options? So, Minimum data collected is first (and the most typical use-case) and then the URL one. Can the Minimum Data collected be checked by default?

Change wording to be "Minimum demographic data (age, sex, age offset) provided directly by participant".

(3) Further on down - Can the "do you want confirmation questions?" question only show up if users select the non-default demographics?

On Wed, Sep 6, 2023 at 2:21 AM Henry Mehta @.***> wrote:

@vmarchman https://github.com/vmarchman I have updated the text/explanation around the two checkboxes. Does this help

— Reply to this email directly, view it on GitHub https://github.com/langcog/web-cdi/issues/472#issuecomment-1707981597, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB2TUTFUC456IMRCFFLU3BLXZA6BBANCNFSM6AAAAAA4HWRSWU . You are receiving this because you were mentioned.Message ID: @.***>

HenryMehta commented 1 year ago

@vmarchman You don't want minimum data to be checked by default. It will mean all the questions about income, birth order, weight etc will not be asked.

I have amended the order, wording etc and the url link only shows once the minimum data item is checked. You'll see what I mean

HenryMehta commented 1 year ago

Video here: https://youtu.be/oGFFlY7PIIY

vmarchman commented 1 year ago

Ahhh. Yes @Henry @.***> Actually, I misspoke I think. I must have been thinking to have a new default checked which is collect all demographic data. But, I think your solution works - i.e., to not have the options about minimum data be available unless you select opt out.

On Wed, Sep 6, 2023 at 11:33 PM Henry Mehta @.***> wrote:

@vmarchman https://github.com/vmarchman You don't want minimum data to be checked by default. It will mean all the questions about income, birth order, weight etc will not be asked.

I have amended the order, wording etc and the url link only shows once the minimum data item is checked. You'll see what I mean

— Reply to this email directly, view it on GitHub https://github.com/langcog/web-cdi/issues/472#issuecomment-1709555307, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB2TUTE3KGQX5K5IFIAZNOTXZFTETANCNFSM6AAAAAA4HWRSWU . You are receiving this because you were mentioned.Message ID: @.***>