Skip to content

[PB-5714] Email base64 #21

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
TamaraFinogina merged 3 commits into from
Jan 15, 2026
Merged

[PB-5714] Email base64 #21

TamaraFinogina merged 3 commits into from
Jan 15, 2026

Conversation

@TamaraFinogina
Copy link
Contributor

@TamaraFinogina TamaraFinogina commented Jan 15, 2026 •
edited
Loading

Description

  • Use string fields and convertion to base64, so encrypted emails can be sent to the server without extra steps.
  • Remove email API as it's moved to sdk

Related Issues

Relates to PB-5714

Checklist

  • Changes have been tested locally.
  • Unit tests have been written or updated as necessary.
  • The code adheres to the repository's coding standards.
  • Relevant documentation has been added or updated.
  • No new warnings or errors have been introduced.
  • SonarCloud issues have been reviewed and addressed.
  • QA Passed

Testing Process

unit tests

@TamaraFinogina TamaraFinogina self-assigned this Jan 15, 2026
@TamaraFinogina TamaraFinogina marked this pull request as ready for review January 15, 2026 15:26
larryrider
larryrider approved these changes Jan 15, 2026
View reviewed changes
Copy link
Contributor

@larryrider larryrider left a comment

Choose a reason for hiding this comment

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

check sonarcloud issues

@TamaraFinogina
Copy link
Contributor Author

TamaraFinogina commented Jan 15, 2026

@larryrider Fixed :)

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 15, 2026

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@TamaraFinogina TamaraFinogina merged commit 4e34d4d into master Jan 15, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@larryrider larryrider larryrider approved these changes

Assignees

@TamaraFinogina TamaraFinogina

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@TamaraFinogina @larryrider