Skip to content

Conversation

@Gavinok
Copy link
Contributor

@Gavinok Gavinok commented Nov 21, 2025

Migrate UA parsing to https://www.npmjs.com/package/my-ua-parser as an api compatible alternative that uses the MIT License

This resolves #910

  • Added my-ua-parser.min.js to replace the outdated UAParser.js in the project.
  • Ensure compatibility and enhanced user-agent parsing capabilities.
  • Updated reference in verified_credentials.html to use the new parser.

Gavinok and others added 2 commits November 21, 2025 11:09
Signed-off-by: Gavin Jaeger-Freeborn <[email protected]>

- Added `my-ua-parser.min.js` to replace the outdated UAParser.js in the project.
- Ensure compatibility and enhanced user-agent parsing capabilities.
- Updated reference in `verified_credentials.html` to use the new parser.
@coveralls
Copy link

coveralls commented Nov 21, 2025

Pull Request Test Coverage Report for Build 19581039137

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 92.397%

Totals Coverage Status
Change from base Build 19575481693: 0.0%
Covered Lines: 1592
Relevant Lines: 1723

💛 - Coveralls

Signed-off-by: Gavin Jaeger-Freeborn <[email protected]>
@Gavinok Gavinok requested review from esune, loneil and swcurran November 21, 2025 21:10
@Gavinok Gavinok merged commit 764228f into openwallet-foundation:main Nov 25, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Need to remove a library with a non-Open Source license

3 participants