[WALL-4040] removed dictionary from rap library

[ghost]

Removed zxcvbn dictionary from deriv-app. Saving 1mb on at least two packages.
Downside is, that we no longer have the "common password" detection.

Gzipped:

[review-deriv]

PR description is too short. You can use this format:

# Description
This is a summary of the changes. This should describe why do we need this change.
Whys and Hows come here.

# Cards
* https://clickup.com/12345
* https://clickup.com/12346

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated (UTC)
deriv-app	✅ Ready (Inspect)	Visit Preview	May 21, 2024 2:05pm

[github-actions]

A production App ID was automatically generated for this PR. (log)

• PR: [WALL-4040] removed dictionary from rap library #15206
• URLs:
  • w/ App ID + Server: https://deriv-app-git-fork-wojciech-deriv-feature-remove-rap-lib-8f207b.binary.sx?qa_server=red.derivws.com&app_id=24202
  • Original: https://deriv-app-git-fork-wojciech-deriv-feature-remove-rap-lib-8f207b.binary.sx
• App ID: 24202

Click here to copy & paste above information.

- **PR**: [https://github.com/binary-com/deriv-app/pull/15206](https://github.com/binary-com/deriv-app/pull/15206)
- **URLs**:
    - **w/ App ID + Server**: https://deriv-app-git-fork-wojciech-deriv-feature-remove-rap-lib-8f207b.binary.sx?qa_server=red.derivws.com&app_id=24202
    - **Original**: https://deriv-app-git-fork-wojciech-deriv-feature-remove-rap-lib-8f207b.binary.sx
- **App ID**: `24202`

[github-actions]

🚨 Lighthouse report for the changes in this PR:

Category	Score
🔺 Performance	25
🟢 Accessibility	90
🟢 Best practices	92
🟧 SEO	81
🟧 PWA	78

Lighthouse ran with https://deriv-app-git-fork-wojciech-deriv-feature-remove-rap-lib-8f207b.binary.sx/

[coveralls]

coverage: 42.505% (+0.04%) from 42.466%
when pulling e732069 on wojciech-deriv:feature/remove-rap-library-vocabulary
into 488058e on binary-com:master.

[review-deriv]

PR description is too short. You can use this format:

# Description
This is a summary of the changes. This should describe why do we need this change.
Whys and Hows come here.

# Cards
* https://clickup.com/12345
* https://clickup.com/12346

[dan-deriv]

What is the improvement for zipped traffic?

[dan-deriv]

@wojciech-deriv should not we do the same for all apps?

[ghost]

@wojciech-deriv should not we do the same for all apps?

hey @dan-deriv , updated PR to remove it from all packages + added screenshots of before/after, normal and gzipped

[sonarqubecloud]

Quality Gate passed

Issues
5 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
5.8% Duplication on New Code

See analysis details on SonarCloud

[github-actions]

⏳ Generating Lighthouse report...

[ghost]

as per technical discoveries, while desired to actually remove it, its only impactfull on certain rare user journeys, so sadly closing for now (sadly, as everyone is waiting to get rid of this and quite a lot of work went there)