Skip to content

Conversation

@DenBond7
Copy link
Collaborator

This PR Improved Password-protected messages compliance

close #2909


Tests (delete all except exactly one):

  • Tests added or updated

To be filled by reviewers

I have reviewed that this PR... (tick whichever items you personally focused on during this review):

  • addresses the issue it closes (if any)
  • code is readable and understandable
  • is accompanied with tests, or tests are not needed
  • is free of vulnerabilities

@DenBond7 DenBond7 marked this pull request as ready for review December 18, 2024 13:22
@DenBond7 DenBond7 requested a review from sosnovsky as a code owner December 18, 2024 13:22
Copy link
Collaborator

@sosnovsky sosnovsky left a comment

Choose a reason for hiding this comment

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

Works well now, thanks for quick fix!

@DenBond7
Copy link
Collaborator Author

Should I make a new release?(1.6.1)

@DenBond7 DenBond7 merged commit 8b78b0a into master Dec 18, 2024
3 checks passed
@DenBond7 DenBond7 deleted the issue_2909_fix_password_protected_messages_compliance_issue branch December 18, 2024 14:07
@sosnovsky
Copy link
Collaborator

Should I make a new release?(1.6.1)

Yes, please create a new release and send updated build to customer, thanks!

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.

Password-protected messages compliance issue

2 participants