Skip to content

refactor: replace native crypto.randomUUID() for better compatibility#648

Merged
chybisov merged 5 commits intomainfrom
replace-crypto-uuid
Jan 28, 2026
Merged

refactor: replace native crypto.randomUUID() for better compatibility#648
chybisov merged 5 commits intomainfrom
replace-crypto-uuid

Conversation

@effie-ms
Copy link
Copy Markdown
Contributor

@effie-ms effie-ms commented Jan 26, 2026

Which Jira task is linked to this PR?

Part of https://lifi.atlassian.net/browse/LF-17240

Why was it implemented this way?

On iOS <15.4, crypto.randomUUID() is not supported: https://caniuse.com/?search=crypto.randomUUID
It is also not supported as a polyfill by core-js, and requires installing some other library instead.
Since it is used only for React keys, and is a single item in the array in any case, it can be replaced with anything simpler.

Checklist before requesting a review

  • I have performed a self-review and testing of my code.
  • This pull request is focused and addresses a single problem.

Docs for other polyfills: https://github.com/lifinance/public-docs/pull/90

@effie-ms effie-ms self-assigned this Jan 26, 2026
@github-actions
Copy link
Copy Markdown

Hey! This is your new endpoint: https://b9fc55c6.widget-replacecry.pages.dev

@effie-ms effie-ms requested a review from chybisov January 26, 2026 15:47
@effie-ms effie-ms marked this pull request as ready for review January 26, 2026 15:47
@effie-ms effie-ms changed the title refactor: replace native crypto.randomUUID() with uuid for better compatibility refactor: replace native crypto.randomUUID() for better compatibility Jan 27, 2026
@github-actions
Copy link
Copy Markdown

Hey! This is your new endpoint: https://683ecef1.widget-replacecry.pages.dev

@chybisov chybisov merged commit b7b46d6 into main Jan 28, 2026
1 check passed
@chybisov chybisov deleted the replace-crypto-uuid branch January 28, 2026 11:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants