-
Notifications
You must be signed in to change notification settings - Fork 141
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
Python library for TON Connect 2.0 #199
Comments
I have almost finished development the TON Connect protocol inside my telegram bot, but it takes several weeks (maybe less) to make it a separate library. |
Closed because not assigned and a new issue has been created. Sad for the wasted time. |
@XaBbl4, we apologize. I’ll give an answer soon. Maybe we’ll reopen it. |
LGTM! |
@XaBbl4, you can continue to perform this footstep. |
We also need to decide the reward. |
@krigga, what do you think about the amount of reward? |
I believe a reward of about $1000 would be fair. |
@XaBbl4, do you agree with the $1000 reward? Ready to do it? |
@delovoyhomie, yes, that's ok. Current status:
Needs do:
|
Added parsing of the result of send transaction and connect event requests. Also added ton_proof check (and example for it). There is also a telegram bot with a demo, but not yet completed. |
Sure, I'll review it |
@XaBbl4 I couldn't properly verify that the library works due to this issue XaBbl4/pytonconnect#1 |
@krigga Thanks. Fixed it. Didn't check this import option, always used |
That one didn't work, too. I'll check the fix soon. |
@krigga, is the issue fixed? What steps are left to complete the footstep? |
@XaBbl4, complete the third milestone. Submit PR to ton-connect repository with a link to the Python library. |
Yes, the library works, I believe the footstep is complete (barring the third milestone). |
@delovoyhomie, Add PR to ton-connect (not without problems, making a couple of unnecessary commits, that don’t know how to remove)
@krigga, I'll try to add it soon. There are a functions that I wanted to complete, also add an example of using the library with a telegram bot, but have not yet found the time to finish it. |
@XaBbl4, I would be very grateful if you could implement a telegram bot to connect the wallet, as it was done in #176.Let's do the same with the bot and documentation in a new footstep once we are done with this. As far as I know, it is possible to revert changes to previous ones (about couple of unnecessary commits) Please try to sort out this problem. |
One commit got into the PR, just a few were added to the Issue history of footstep. I did't work with github so often, more with gitlab. |
@XaBbl4, could you please let me know here when the PR to ton-connect-list has been merged? |
@krigga, publish v0.2.0: added method
@delovoyhomie, something I doubt that the PR will be accepted quickly, there is another PR dated March 30 which remained without discussion. |
@XaBbl4 the new features work, thank you. Could you please remove these specifiers https://github.com/XaBbl4/pytonconnect/blob/5e860b52359a43919605f25239f96467724f07f8/setup.cfg#L15-L16 in the manifest? Python 3.8 cannot handle the type hints like |
@XaBbl4, please send me your wallet address to receive the reward. |
@krigga, thx. To keep compatibility with Python 3.7 and 3.8, the type hint changed to @delovoyhomie, my wallet |
Rewards sent! Thank you for the contribution. |
Summary
At this step, it is proposed to create a library for working with the TON Connect protocol in Python.
Context
At the moment, there is only one library for working with the TON Connect protocol - written in JS. Need libraries in other languages to reach as many developers and improve the quality of dApps.
For the first version, need to add support http-bridge-compatible wallets.
Goals
Deliverables
Definition of Done
Reward
Total: $1000
The text was updated successfully, but these errors were encountered: