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

Update substrate/connect library to 2.0.1 #2588

Open
stepanLav opened this issue Nov 7, 2024 · 3 comments
Open

Update substrate/connect library to 2.0.1 #2588

stepanLav opened this issue Nov 7, 2024 · 3 comments
Assignees
@Asmadek
Labels
Clarification required Issues require more info or research Enhancement Added to issues and PRs when improving how feature works but without significant changes Ready for dev Ready for development Tier-2
Milestone
Milestone 4

Comments

@stepanLav
Copy link
Member

stepanLav commented Nov 7, 2024 •
edited by pgolovkin
Loading

Description

New version of substrate/connect is available to use - https://github.com/paritytech/substrate-connect/releases/tag/%40substrate%2Fconnect%402.0.1

Additional Information

Substrate connect demo https://github.com/paritytech/substrate-connect/blob/main/projects/demo/src/index.ts

More examples https://github.com/paritytech/substrate-connect/tree/main/projects

Seems that how connection is built in the code should be reworked.

Need to check:

  • compatibilities
  • performance
  • web\desktop versions (also usage with substrate-connect web extension)

Priority

[Medium]
@stepanLav stepanLav added the Feedback Issues & PRs addressing users' feedback label Nov 7, 2024
@stepanLav stepanLav added Enhancement Added to issues and PRs when improving how feature works but without significant changes and removed Feedback Issues & PRs addressing users' feedback labels Nov 7, 2024
@stepanLav stepanLav changed the title Update smoldot library to 2.0.1 Update substrate/connect library to 2.0.1 Nov 7, 2024
@stepanLav stepanLav mentioned this issue Nov 7, 2024
Open
@pgolovkin
Copy link
Collaborator

pgolovkin commented Nov 7, 2024 •
edited
Loading

It's better to test on the web build with substrate-connect browser extension.

When the provider is connected according substrate-connect browser extension, Nova Spektr shows that it isn't connected (video is attached).

Something should be changed in the code that checks the connection status for Light client providers.

Seems that the main problem is here in the code.

Screen.Recording.2024-11-07.at.11.29.42.AM.mov

@pgolovkin pgolovkin moved this from Todo to Development in Nova Spektr development Nov 7, 2024
@pgolovkin pgolovkin moved this from Development to Todo in Nova Spektr development Nov 14, 2024
@pgolovkin pgolovkin removed their assignment Nov 14, 2024
@pgolovkin pgolovkin added the Ready for dev Ready for development label Nov 19, 2024
@pgolovkin pgolovkin modified the milestones: Milestone 5, Milestone 4 Nov 20, 2024
@Asmadek Asmadek moved this from Todo to Development in Nova Spektr development Dec 23, 2024
@pgolovkin pgolovkin moved this from Development to Todo in Nova Spektr development Dec 25, 2024
@Asmadek
Copy link
Contributor

Asmadek commented Dec 25, 2024

Add issue in substrate connect lib

@pgolovkin
Copy link
Collaborator

Need to discuss if we need it right now, or migration to PAPI is better?

@pgolovkin pgolovkin added the Clarification required Issues require more info or research label Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Asmadek Asmadek

Labels
Clarification required Issues require more info or research Enhancement Added to issues and PRs when improving how feature works but without significant changes Ready for dev Ready for development Tier-2
Projects
Nova Spektr development
Status: Todo
Milestone
Milestone 4
Development

No branches or pull requests
3 participants
@pgolovkin @Asmadek @stepanLav