Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enhancements to Notification System and Fixes #51

Merged
merged 20 commits into from
Feb 27, 2025

Conversation

71walceli
Copy link
Collaborator

This pull request includes significant updates to the configuration and functionality of the identity registrar component, as well as improvements to the handling of real-time chain information and WebSocket connections.

Configuration Updates:

  • Added dynamic configuration for local testnets, enabling them based on environment variables. (src/api/config.ts) [1] [2]

Identity Registrar Component Enhancements:

  • Introduced a notification system using an Accordion component to display alerts. (src/components/identity-registrar.tsx) (Fa89c9d5L147R183)
  • Improved transaction handling by fetching the nonce dynamically and preventing duplicate notifications. (src/components/identity-registrar.tsx) [1] [2]
  • Added logic to clear alerts when the chain or account changes. (src/components/identity-registrar.tsx)
  • Enhanced the judgement notification to differentiate between successful and unsuccessful identity verifications. (src/components/identity-registrar.tsx)

Real-Time Chain Information:

  • Refactored the useChainRealTimeInfo hook to use the System.Events API for more efficient event handling. (src/hooks/useChainRealTimeInfo.ts) [1] [2]

WebSocket Connection Enhancements:

  • Extended the useChallengeWebSocket hook to include methods for subscribing, connecting, and disconnecting from the WebSocket. (src/hooks/useChallengeWebSocket.ts) [1] [2]

71walceli added 19 commits February 18, 2025 17:39
- Will now listen at best blocks, having faster state updates.
- Refactor icon handling in IdentityForm and StatusPage
@hitchhooker hitchhooker merged commit 0b92e4c into main Feb 27, 2025
1 check 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.

2 participants