- First commit to issue fork.
- Merge request !9MR of patch #3 https://www.drupal.org/project/encrypt/issues/2984719#comment-12697611 → (Open) created by ptmkenny
- 🇯🇵Japan ptmkenny
Made an MR of patch #3 so that this can be tested against GitLab CI.
Instead of providing a default encryption method, let's update the description on the form when no encryption method is present to make sure the user is pointed to the docs (which I've recently migrated to the d8 handbook here: https://www.drupal.org/docs/8/modules/encrypt → ).
The way I've got in the patch will update the description. The other method I toyed with is replacing the select element entirely with just a markup that has the explanation.
2 things to consider:
1. Right now encryption method isn't a required field, should we require this so the form errors if nothing is selected?
2. Should we do the same for the keys, explaining how to create secure keys (mentioned in the issue here:
https://www.drupal.org/project/encrypt/issues/2863424 →
)
Patch incoming
Needs review
3.0
Code
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
Made an MR of patch #3 so that this can be tested against GitLab CI.