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.

Sign up for GitHub

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

Jump to bottom

Move polling of the network health and status inside the respective classes. #2362

Closed
Tracked by #2323
Tuditi opened this issue Apr 24, 2024 · 0 comments · Fixed by #2379
Closed
Tracked by #2323

Move polling of the network health and status inside the respective classes. #2362

Tuditi opened this issue Apr 24, 2024 · 0 comments · Fixed by #2379
Assignees
@MarkNerdi @Tuditi
Labels
type:enhancement
Milestone
E02.M07 - Multi-I...

Comments

@Tuditi
Copy link
Contributor

Tuditi commented Apr 24, 2024 •
edited
Loading

Move polling and destruction of polling into the respective network classes.
BaseNetwork should be extended with a startPoll & destroy() function that cleans up all polls and state upon logout.

This function is then implemented for the Stardust and Evm chains.

The same should happen for getting the network status.
@Tuditi Tuditi mentioned this issue Apr 24, 2024
Closed
11 tasks
@Tuditi Tuditi moved this from 📋 Backlog to 🔖 Todo in Bloom - Task Management Apr 24, 2024
@Tuditi Tuditi moved this from 🔖 Todo to 🏗 In Progress in Bloom - Task Management Apr 26, 2024
@Tuditi Tuditi mentioned this issue Apr 26, 2024
Merged
10 tasks
@Tuditi Tuditi linked a pull request Apr 26, 2024 that will close this issue
chore: move network health into class #2379
Merged
10 tasks
@Tuditi Tuditi moved this from 🏗 In Progress to 👀 Needs Review in Bloom - Task Management Apr 28, 2024
@github-project-automation github-project-automation bot moved this from 👀 Needs Review to 🎉 Done in Bloom - Task Management Apr 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@MarkNerdi MarkNerdi

@Tuditi Tuditi

Labels
type:enhancement
Projects
Bloom - Task Management
Archived in project
Milestone
E02.M07 - Multi-ISC Support
Development

Successfully merging a pull request may close this issue.

chore: move network health into class bloomwalletio/bloom
3 participants
@MarkNerdi @Tuditi @nicole-obrien