- Issue created by @eelkeblok
- Status changed to Needs review
over 1 year ago 10:16am 3 November 2023 - π³π±Netherlands eelkeblok Netherlands π³π±
Submitting for review, although I suppose this needs tests. Suggestions what/how to test welcome.
- Merge request !38Catch EncryptException in case of problems encrypting/decrypting β (Open) created by eelkeblok
- last update
over 1 year ago 36 pass - Status changed to RTBC
12 months ago 7:05am 25 April 2024 - π³π±Netherlands ahmetburkan Utrecht
Tested the functionality and it works as expected! There is an error notification when the decrypting fails and when encrypting secure field data fails.
I agree with eelkeblok β about the testing part. For the sake of getting this in the codebase i will set the status to RTBC. - Status changed to Needs work
11 months ago 1:44am 18 May 2024 - π―π΅Japan ptmkenny
If it βneeds testsβ it cannot be RTBC. The code is committed with the tests if it needs tests.
- First commit to issue fork.
- Merge request !61Catch EncryptException in case of problems encrypting/decrypting in 3.2 β (Open) created by hmdnawaz
- π³π±Netherlands eelkeblok Netherlands π³π±
It is unclear to me whether !61 is just a rebase or whether there are other differences to !38. I updated !38 by merging in 3.2.x and resolving conflicts.