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

#306 dex swap by @ton.md #456

Merged
merged 7 commits into from
Feb 1, 2024
Merged

#306 dex swap by @ton.md #456

merged 7 commits into from
Feb 1, 2024

Conversation

mbaneshi
Copy link
Contributor

No description provided.

@Gusarich
Copy link
Member

@mbaneshi Thank you for the contribution!
However, let's fix some minor issues with it first.

  • you use sender variable multiple times but don't show how to get it
  • there are lots of little mistakes with letter registers, spaces etc. would be nice to fix them too

@mbaneshi
Copy link
Contributor Author

@Gusarich Thanks 🙏.
Sure , I will revise and correct them.

@mbaneshi
Copy link
Contributor Author

Hi @Gusarich
I think, it is done, would you please check it? Thank you.

@Gusarich
Copy link
Member

might need some more polishing, but I think we can merge it now

@reveloper
Copy link
Member

@mbaneshi, @Gusarich, Thank you!

@reveloper reveloper merged commit 47e3b8d into ton-community:main Feb 1, 2024
1 check passed
@mbaneshi
Copy link
Contributor Author

mbaneshi commented Feb 1, 2024

@reveloper thank you too, this is my first merge contribution, and I am excited

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.

3 participants