Skip to content

[PB-4638] Remove crypto js #321

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
Open
CandelR wants to merge 2 commits into
base: master
Choose a base branch
from
Open

[PB-4638] Remove crypto js #321

CandelR wants to merge 2 commits into from

Conversation

@CandelR
Copy link
Contributor

@CandelR CandelR commented Oct 16, 2025 •
edited
Loading

  • Removed crypto-js lib and related code

Note:

  • The UI for the Change Password feature has been left commented out in case it is reused when the flow is redesigned

@CandelR CandelR self-assigned this Oct 16, 2025
@CandelR CandelR added the enhancement New feature or request label Oct 16, 2025
@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 16, 2025

Quality Gate Passed Quality Gate passed

Issues
6 New issues
0 Accepted issues

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

See analysis details on SonarQube Cloud

@CandelR CandelR marked this pull request as ready for review October 16, 2025 08:44
Base automatically changed from feature/PB-5113-replace-login-with-web-login to release/1.8.3 October 16, 2025 08:54
Base automatically changed from release/1.8.3 to master October 24, 2025 12:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@CandelR CandelR

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@CandelR