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

Add Member tiankonglan #64

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Conversation

tiankonglan
Copy link

No description provided.

@auto-assign auto-assign bot requested a review from nkysg July 25, 2022 15:42
@nkysg
Copy link
Contributor

nkysg commented Jul 27, 2022

Hello @tiankonglan , welcome to the StarTrek! You can take a look at the Cookbook first, there is a lot of Starcoin knowledge in it, if you are having trouble learning about Starcoin, you can send me a private message by this way or discuss if in a development group.

@github-actions
Copy link

Welcome to the Web3 StarTrek 🎉!

You can take a look at the Starcoin Cookbook first, there is a lot of Starcoin knowledge in it.
If you are having trouble learning about Starcoin, you can ask and discuss in the development group.

@tiankonglan
Copy link
Author

@leeduckgo hi, I have learn move, and develop a did contract

@leeduckgo leeduckgo requested review from leeduckgo and removed request for nkysg October 7, 2022 04:36
@tiankonglan
Copy link
Author

@nkysg @jolestar I have add more detail info for apply member

https://mp.weixin.qq.com/s/LBvvz3OffYQysL27MnBw0g
## 学习成果
master move contract code in both starcoin and aptos
https://github.com/NonceGeek/DID-Solution-In-Move/did
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sorry, I have fix it.

@jolestar
Copy link
Contributor

jolestar commented Oct 8, 2022

cc @jiangying000 @WGB5445


### 新项目:
- impl aptos did contract add multi sign type
https://github.com/NonceGeek/DID-Solution-In-Move/did-aptos
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sorry, I have fix it.

Copy link
Member

@jiangying000 jiangying000 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe L2 graduation

I am not sure about: https://github.com/NonceGeek/DID-Solution-In-Move/tree/main/did-aptos , since it is not a starcoin repo and I did not find any commit history associated to @tiankonglan

@jolestar
Copy link
Contributor

jolestar commented Oct 8, 2022

🎉 Graduated with Level 2

But the DAO balance is not enough, please wait for new funds to come in.

@leeduckgo
Copy link
Contributor

Congratulation!

@tiankonglan
Copy link
Author

tiankonglan commented Oct 9, 2022

🎉 Graduated with Level 2

But the DAO balance is not enough, please wait for new funds to come in.

doesn't matter

Maybe L2 graduation

I am not sure about: https://github.com/NonceGeek/DID-Solution-In-Move/tree/main/did-aptos , since it is not a starcoin repo and I did not find any commit history associated to @tiankonglan

you can see the pull request record, I commit serverl pr.

@tiankonglan
Copy link
Author

🎉 Graduated with Level 2

But the DAO balance is not enough, please wait for new funds to come in.

doesn't matter.

@tiankonglan
Copy link
Author

Congratulation!

Thanks very much

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.

7 participants