fix: adjust Ledger signatures for notifications

[iamacook]

What it solves

Resolves Ledger notification registration not working

How this PR fixes it

Ledger produces vrs signatures with a canonical v value of {0,1}, while Ethereum's ecrecover call only accepts a non-standard v value of {27,28}. More on that here.

The v of Ledger signatures is now adjusted by 27 accordingly.

How to test it

• Register for notifications with a non-Ledger wallet and observe registration working.
• Register for notifications with a Ledger and observe registration working.

Checklist

• I've tested the branch on mobile 📱
• I've documented how it affects the analytics (if at all) 📊
• I've written a unit/e2e test for it (if applicable) 🧑‍💻

[github-actions]

Branch preview

✅ Deploy successful!

https://fix_ledger_notification_signature--walletweb.review-wallet-web.5afe.dev

[github-actions]

ESLint Summary View Full Report

Annotations are provided inline on the Files Changed tab. You can also see all annotations that were generated on the annotations page.

Type	Occurrences	Fixable
Errors	0	0
Warnings	0	0
Ignored	0	N/A

• Result: ✅ success
• Annotations: 0 total

---

^{Report generated by eslint-plus-action}

[usame-algan]

Looks good 👍

[francovenica]

Looks good.

I can sign the notification enabling with ledger
I get the notifications after the correspondant actions (receiving tokens, excuting tx...)
I tried with regular MM wallet and it works fine too.