Skip to content

Conversation

shainaraskas
Copy link
Collaborator

@shainaraskas shainaraskas commented Mar 24, 2025

made the security overview page much shorter

pages to review:

and some small style polish things

todo:

@shainaraskas shainaraskas requested a review from eedugon March 24, 2025 18:49
Copy link
Contributor

@eedugon eedugon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Feels very good! I've just added some comments for the encryption at rest page, which still looks a bit confusing.

And in a separate thread I'll share feedback about the minimal, basic and basic + HTTPS concepts, because they were not part of this PR.

shainaraskas and others added 2 commits March 25, 2025 12:49
@shainaraskas shainaraskas requested a review from eedugon March 25, 2025 16:56
Copy link
Contributor

@eedugon eedugon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!!!

@eedugon eedugon merged commit c1d6208 into main Mar 26, 2025
4 checks passed
@eedugon eedugon deleted the security-cleanup branch March 26, 2025 07:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants