Add encrypt dict using property decorator

benjamin-awd commented 8 months ago

Description of the changes

Describe the changes in this PR, with references to issues in the issue tracker and forum discussions (if applicable).

Caveats

If the implementation has "sharp edges" or is otherwise incomplete, please explain here. Deviations from the contribution guidelines should also be motivated in this section.

In particular, if your contribution includes any of the following, please provide a brief justification here:

Changes to project dependencies.
Changes to existing public APIs (particularly if they could break existing user code) or the CLI.
Nontrivial changes to internal (non-public) APIs

Checklist

Please go over this checklist to increase the chances of your PR being worked on in a timely manner. Deviations are allowed with proper justification (see previous section).

[ ] I have read the project's CoC and contribution guidelines.
[ ] I understand and agree to the terms in the Developer Certificate of Origin as applied to this contribution.
[ ] All new code in this PR has full test coverage.

For new features (delete if not applicable)

[ ] I have discussed the implementation of this feature with the project maintainer(s) on the discussion forum or over email.
[ ] I have verified that my changes do not break existing API or CLI functionality, or ensured that all breaking changes are clearly documented in this PR.
[ ] All public API functionality in this PR is documented.

For bug fixes (delete if not applicable)

[ ] My changes do not affect any public API or CLI semantics.
[ ] My PR adds regression tests (i.e. tests that fail if the bug fix is not applied).
[ ] All new code in this PR has full test coverage.

For documentation contributions (delete if not applicable)

[ ] I have built the HTML documentation locally, and verified that the changes behave correctly in-browser.

Additional comments

Feel free to add any additional comments here.

MatthiasValvekens commented 8 months ago

Did you see #343? :)

EDIT: oops, didn't spot that this was a diff against that branch. Was replying on mobile at the time.

benjamin-awd commented 8 months ago

Did you see https://github.com/MatthiasValvekens/pyHanko/pull/343? :)

Ah yes I did -- just raising this as a diff, feel free to disregard this PR completely

MatthiasValvekens / pyHanko