feat(dApp): Added Git identity binding during member registration. #224

yhoungdev · 2025-07-14T16:41:19Z

Adds SEP-53 compliant Git identity linking during registration. Supports GitHub/GitLab with Ed25519 signature verification.

Optional provider:username input
Auto-fetch public keys from Git providers
SEP-53 envelope with network validation
Signature verification using @noble/ed25519
Step-by-step UI flow with skip option

This PR fixed the issue on #217

netlify · 2025-07-14T16:41:24Z

✅ Deploy Preview for staging-tansu ready!

Name	Link
🔨 Latest commit	`7cce6c1`
🔍 Latest deploy log	https://app.netlify.com/projects/staging-tansu/deploys/68eb27be777b1900086dfa93
😎 Deploy Preview	https://deploy-preview-224--staging-tansu.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.
Lighthouse	1 paths audited Performance: 69 Accessibility: 89 Best Practices: 83 SEO: 92 PWA: 80 View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

netlify · 2025-07-14T16:42:39Z

✅ Deploy Preview for tansu canceled.

Name	Link
🔨 Latest commit	`0d0ca64`
🔍 Latest deploy log	https://app.netlify.com/projects/tansu/deploys/687533329b64c00008b0d26b

tupui

Good start thanks! A few comments.

dapp/src/components/page/dashboard/MemberProfileModal.tsx

tupui · 2025-07-20T00:35:03Z

dapp/packages/tansu/src/index.ts

Actually, I am not sure we need all that to be in the contract itself vs the meta file-if even needed. I would keep the handle and key only. And in meta nothing. What do you think?

Sorry i don't get you clearly

you mean i should remove this git_identity?: string; git_pubkey?: Buffer; msg?: string; sig?: Buffer; signed_at?: u64;

?

Yes we just need the identity and key to be stored in the contract. When I will call the smart contract function add_member I will pass everything though as to validate on-chain the data.

tupui · 2025-09-30T11:01:03Z

@yhoungdev are you still working on this?

yhoungdev · 2025-09-30T11:14:39Z

Yes i am @tupui

yhoungdev · 2025-09-30T11:14:52Z

would resolve the conflict in a little while

netlify · 2025-10-03T06:38:43Z

✅ Deploy Preview for tansu canceled.

Name	Link
🔨 Latest commit	`7cce6c1`
🔍 Latest deploy log	https://app.netlify.com/projects/tansu/deploys/68eb27be8967570008f6c440

tupui · 2025-10-03T09:56:41Z

Is this ready for another review? Or you still need to update to integrate what I had said here and in our chat?

tupui · 2025-10-03T17:10:42Z

@yhoungdev let me know.

yhoungdev · 2025-10-03T20:25:51Z

It is still a wip in my branch.

…into feature/git_identity_binding

yhoungdev · 2025-10-05T16:43:20Z

@yhoungdev are you still working on this?

You can check again

tupui · 2025-10-05T17:15:42Z

@yhoungdev please test first yourself the deployment https://deploy-preview-224--staging-tansu.netlify.app/

yhoungdev · 2025-10-05T17:21:36Z

Ok

yhoungdev · 2025-10-05T19:56:36Z

@tupui from your test how was it so far?

tupui · 2025-10-05T20:18:12Z

I asked you above to check the deployment because the deployment is not working.

JoinCommunityModal.TaDYHw7a.js:2 Uncaught ReferenceError: GitVerification is not defined
    at ye (JoinCommunityModal.TaDYHw7a.js:2:6319)
    at jf (client.CH8PIGmd.js:32:47866)
    at nc (client.CH8PIGmd.js:32:70573)
    at qy (client.CH8PIGmd.js:32:80897)
    at fv (client.CH8PIGmd.js:32:116458)
    at xm (client.CH8PIGmd.js:32:115538)
    at _c (client.CH8PIGmd.js:32:115375)
    at lv (client.CH8PIGmd.js:32:112211)
    at Tv (client.CH8PIGmd.js:32:123862)
    at MessagePort.Ba (client.CH8PIGmd.js:9:1643)

yhoungdev · 2025-10-05T20:45:43Z

ok, but the Irony here is that Netlify is building fine 😅

yhoungdev · 2025-10-05T20:45:58Z

I asked you above to check the deployment because the deployment is not working.

JoinCommunityModal.TaDYHw7a.js:2 Uncaught ReferenceError: GitVerification is not defined
    at ye (JoinCommunityModal.TaDYHw7a.js:2:6319)
    at jf (client.CH8PIGmd.js:32:47866)
    at nc (client.CH8PIGmd.js:32:70573)
    at qy (client.CH8PIGmd.js:32:80897)
    at fv (client.CH8PIGmd.js:32:116458)
    at xm (client.CH8PIGmd.js:32:115538)
    at _c (client.CH8PIGmd.js:32:115375)
    at lv (client.CH8PIGmd.js:32:112211)
    at Tv (client.CH8PIGmd.js:32:123862)
    at MessagePort.Ba (client.CH8PIGmd.js:9:1643)

Ok, i will fix this.

tupui · 2025-10-09T15:57:06Z

Any updates? Just so you know. OnlyDust is soon ending all their programs. If you don't finish very soon, I won't be able to give you anything with their platform.

yhoungdev · 2025-10-09T16:15:34Z

Yess i have an update.

yhoungdev · 2025-10-09T23:22:33Z

Conflict has been resolved.

yhoungdev · 2025-10-09T23:28:44Z

I asked you above to check the deployment because the deployment is not working.

JoinCommunityModal.TaDYHw7a.js:2 Uncaught ReferenceError: GitVerification is not defined
    at ye (JoinCommunityModal.TaDYHw7a.js:2:6319)
    at jf (client.CH8PIGmd.js:32:47866)
    at nc (client.CH8PIGmd.js:32:70573)
    at qy (client.CH8PIGmd.js:32:80897)
    at fv (client.CH8PIGmd.js:32:116458)
    at xm (client.CH8PIGmd.js:32:115538)
    at _c (client.CH8PIGmd.js:32:115375)
    at lv (client.CH8PIGmd.js:32:112211)
    at Tv (client.CH8PIGmd.js:32:123862)
    at MessagePort.Ba (client.CH8PIGmd.js:9:1643)

Ok, i will fix this.

This has been fixed, please check and confirm.

yhoungdev · 2025-10-09T23:28:58Z

Let me know if there are any other issues/errors.

yhoungdev · 2025-10-10T02:23:47Z

Can i see error logs? if there is any.

tupui · 2025-10-10T03:26:54Z

You can see the logs on Netlify yes. Just look at the links for the CI.

Otherwise did you test it with the deployment now? Can you share screenshots of the tests?

yhoungdev · 2025-10-10T04:09:02Z

https://deploy-preview-224--staging-tansu.netlify.app/

After "link Git handle" button has been clicked

yhoungdev · 2025-10-12T04:08:23Z

I just pushed a fix for it @tupui

for the now i am using an external proxy server with the Github endpoint

So technically it will be like this
https://corsproxy.io/?${encodeURIComponent(originalUrl)}

Now you can now make requests to fetch pkeys safely, let me know if there are any issues.

tupui · 2025-10-16T15:10:31Z

@yhoungdev I am trying to test but got some issues

Also for the command to type, this is not portable. I think we don't need the /dev/stdin since it's a pipe and use the last command's output as an input.

yhoungdev · 2025-10-16T15:12:02Z

Ok, no problem, i will look into that fix @tupui

tupui · 2025-11-30T18:01:37Z

@yhoungdev are you still interested in finishing this one?

yhoungdev · 2025-11-30T18:10:03Z

Yes Roy, i would attend to the error in some hours.

yhoungdev · 2025-11-30T18:34:18Z

Looking into the conflict.

feat (Git): Added Git identity binding during member registration.

0d0ca64

tupui reviewed Jul 20, 2025

View reviewed changes

tupui linked an issue Sep 25, 2025 that may be closed by this pull request

Add Git identity binding during member registration #217

Open

fixed merge conflict

feb9653

yhoungdev added 2 commits October 5, 2025 15:31

Merge branch 'main' of https://github.com/yhoungdev/soroban-versioning …

e4e92de

…into feature/git_identity_binding

chore: updated suggestions

525d223

fixing conflict

d892caa

fixed undefined Gitverevification function error

62e5f3b

bug: fixed Github binding CORs issues

7cce6c1

feat(dApp): Added Git identity binding during member registration. #224

Are you sure you want to change the base?

feat(dApp): Added Git identity binding during member registration. #224

Uh oh!

Conversation

yhoungdev commented Jul 14, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

netlify bot commented Jul 14, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✅ Deploy Preview for staging-tansu ready!

Uh oh!

netlify bot commented Jul 14, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✅ Deploy Preview for tansu canceled.

Uh oh!

tupui left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

tupui Jul 20, 2025

Choose a reason for hiding this comment

Uh oh!

yhoungdev Jul 20, 2025

Choose a reason for hiding this comment

Uh oh!

tupui Jul 20, 2025

Choose a reason for hiding this comment

Uh oh!

tupui commented Sep 30, 2025

Uh oh!

yhoungdev commented Sep 30, 2025

Uh oh!

yhoungdev commented Sep 30, 2025

Uh oh!

netlify bot commented Oct 3, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✅ Deploy Preview for tansu canceled.

Uh oh!

tupui commented Oct 3, 2025

Uh oh!

tupui commented Oct 3, 2025

Uh oh!

yhoungdev commented Oct 3, 2025

Uh oh!

yhoungdev commented Oct 5, 2025

Uh oh!

tupui commented Oct 5, 2025

Uh oh!

yhoungdev commented Oct 5, 2025

Uh oh!

yhoungdev commented Oct 5, 2025

Uh oh!

tupui commented Oct 5, 2025

Uh oh!

yhoungdev commented Oct 5, 2025

Uh oh!

yhoungdev commented Oct 5, 2025

Uh oh!

tupui commented Oct 9, 2025

Uh oh!

yhoungdev commented Oct 9, 2025

Uh oh!

yhoungdev commented Oct 9, 2025

Uh oh!

yhoungdev commented Oct 9, 2025

Uh oh!

yhoungdev commented Oct 9, 2025

Uh oh!

yhoungdev commented Oct 10, 2025

Uh oh!

tupui commented Oct 10, 2025

Uh oh!

yhoungdev commented Oct 10, 2025

Uh oh!

yhoungdev commented Oct 12, 2025

Uh oh!

tupui commented Oct 16, 2025

Uh oh!

yhoungdev commented Oct 16, 2025

yhoungdev commented Jul 14, 2025 •

edited

Loading

netlify bot commented Jul 14, 2025 •

edited

Loading

netlify bot commented Jul 14, 2025 •

edited

Loading

netlify bot commented Oct 3, 2025 •

edited

Loading