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.

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

Fix encrypted folder icon #11756

Merged
merged 1 commit into from
Jun 20, 2023
Merged

Fix encrypted folder icon #11756

merged 1 commit into from
Jun 20, 2023

Conversation

ZetaTom
Copy link
Collaborator

@ZetaTom ZetaTom commented Jun 20, 2023

Previously, if one selected Set as encrypted without having set up end-to-end encryption beforehand, the setup dialogue would appear. After completing the setup, the folder would open. However, when returning to the main menu, the folder would not be shown as encrypted until the view was manually refreshed. This change addresses that behaviour.


  • Tests written, or not not needed

@github-actions
Copy link

Codacy

Lint

TypemasterPR
Warnings7676
Errors00

SpotBugs

CategoryBaseNew
Bad practice2626
Correctness8383
Dodgy code435435
Experimental22
Internationalization99
Malicious code vulnerability22
Multithreaded correctness99
Performance5656
Security1818
Total640640

@github-actions
Copy link

APK file: https://www.kaminsky.me/nc-dev/android-artifacts/11756.apk

qrcode

To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.

@github-actions
Copy link

@nextcloud-android-bot
Copy link
Collaborator

@nextcloud-android-bot
Copy link
Collaborator

@AndyScherzinger AndyScherzinger added this to the Nextcloud App 3.26.0 milestone Jun 20, 2023
@AndyScherzinger AndyScherzinger merged commit e70e262 into master Jun 20, 2023
17 of 19 checks passed
@delete-merged-branch delete-merged-branch bot deleted the fix/encrypted-folder-icon branch June 20, 2023 11:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants