Fix: Handle case when browser does not support notifications and ensure new VC display - Githubissues

wwWallet / wallet-frontend

BSD 2-Clause "Simplified" License

24 stars 7 forks source link

Fix: Handle case when browser does not support notifications and ensure new VC display #316

Closed gkatrakazas closed 2 months ago

gkatrakazas commented 2 months ago

Added check for support of the browser Notification API
Implement polling to detect new VCs when browser notifications are not supported.
Enhance logic to properly display new VCs even without notification support.
Refactor CredentialsContext for better code organization, reducing duplication, and improving maintainability.
Make sure display new VCs css animation only if we find new one.

Test it:

Use Safari on iOS, which does not support the Notification API, or disable notification permissions.
Try to obtain a new VC and verify that it is detected and displayed correctly with animation.