Skip to content

chore: replace uuid with native crypto#134

Open
Roman (gameroman) wants to merge 1 commit into
dfpc-coe:mainfrom
gameroman:replace-uuid
Open

chore: replace uuid with native crypto#134
Roman (gameroman) wants to merge 1 commit into
dfpc-coe:mainfrom
gameroman:replace-uuid

Conversation

@gameroman

Copy link
Copy Markdown
Contributor

No description provided.

@ingalls

Nick (ingalls) commented Jun 26, 2026

Copy link
Copy Markdown
Member

Once again thanks a million for the PR!

Unfortunately I'll have to do some more testing to ensure I can merge this. This is currently partially used in a browser context as well which only supports crypto in secure contexts. I need to make sure this includes a CapacitorJS created WebView

@ingalls Nick (ingalls) added the enhancement New feature or request label Jun 26, 2026
@ingalls Nick (ingalls) self-assigned this Jun 26, 2026
@codecov

codecov Bot commented Jun 26, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 69.23077% with 4 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
lib/builders/chat.ts 66.66% 1 Missing ⚠️
lib/cot.ts 0.00% 1 Missing ⚠️
lib/data-package.ts 75.00% 1 Missing ⚠️
lib/parser/normalize_geojson.ts 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants