-
Notifications
You must be signed in to change notification settings - Fork 7
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
UPDATE:Transfer Function #7
Conversation
@morelucks I think you need to pull from main and sync your implementation with the existing codebase. Good work. |
OK thanks
…On Fri, Dec 13, 2024, 1:57 PM CodingCas ***@***.***> wrote:
@morelucks <https://github.com/morelucks> I think you need to pull from
main and sync your implementation with the existing codebase. Good work.
—
Reply to this email directly, view it on GitHub
<#7 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AOQ6DYI7SQ7PCNT6UWDLQ4L2FLKSZAVCNFSM6AAAAABTRUH6BWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNBRGQYDQMRTGQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
@morelucks The code looks good but it seems you did not remove the node modules from the commit history, please do so and also remove the package-lock.json file from the commit history, its an auto generated file. |
then please you might have to run forge fmt so that the CI/CD workflow can run successfully |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
chnages approved
This pull request introduces the transfer function, enabling the transfer of USDT from one user to another using World ID for identity verification. and the test cases