Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Make the Encryption optional #316

Open
casaout opened this issue Sep 14, 2024 · 1 comment · Fixed by #329
Open

Make the Encryption optional #316

casaout opened this issue Sep 14, 2024 · 1 comment · Fixed by #329
Assignees
@SRichner
Labels
enhancement

Comments

@casaout
Copy link
Member

casaout commented Sep 14, 2024

As we are still working on a guide to analyze the encrypted database files, I'd like to make this feature optional (disabled by default) so the researcher can set it in the study.config.ts file.

Changes:

  • Please add a property dataExportEncrypted (default: false)
  • Update the exporter to consider the property
  • Hide the text on the last page of the exporter about the encryption
SRichner added a commit that referenced this issue Oct 21, 2024
@SRichner
[#316] Add study config prop dataExportEncrypted
1001f35
SRichner added a commit that referenced this issue Oct 21, 2024
@SRichner
[#316] Make data export encryption based on parameter
50778a6
SRichner added a commit that referenced this issue Oct 21, 2024
@SRichner
[#316] Update UI to show hints based on encryption config
a75d950
@SRichner SRichner linked a pull request Oct 21, 2024 that will close this issue
316 make the encryption optional #329
Merged
SRichner added a commit that referenced this issue Oct 22, 2024
@SRichner
[#316] Fix description in data export
128bd46
SRichner added a commit that referenced this issue Oct 22, 2024
@SRichner
[#316] Update privacy.md
b9d28dd
SRichner added a commit that referenced this issue Oct 22, 2024
@SRichner
[#316] Update text
442b6ab
@casaout casaout mentioned this issue Oct 24, 2024
Open
@casaout casaout reopened this Oct 31, 2024
@casaout
Copy link
Member Author

casaout commented Oct 31, 2024

@SRichner in the newest release (v0.0.22), the export is no longer encrypted (correct, as per the config), but the text is not completely updated, as it still shows the following sentence:

Image

@casaout casaout removed their assignment Oct 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@SRichner SRichner

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

316 make the encryption optional HASEL-UZH/PersonalAnalytics
2 participants
@casaout @SRichner