Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Feat: Integrating Argent Web Wallet SDk for Telegram mini-app #562

Conversation

emarc99
Copy link
Contributor

@emarc99 emarc99 commented Feb 14, 2025

Closes: #

@emarc99
Copy link
Contributor Author

emarc99 commented Feb 14, 2025

image
Still configuring the SDK appropriately to handle transaction signing and session connection when application is visited with mobile. The above picture is a work in progress

@emarc99 emarc99 marked this pull request as draft February 14, 2025 16:37
@djeck1432 djeck1432 self-requested a review February 16, 2025 10:01
>
<span>Connect Wallet</span>
<span>Connect Wallet - Mobile</span>
Copy link
Owner

Choose a reason for hiding this comment

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

Not mobile, but Telegram

const provider = new RpcProvider({});

const argentWebWallet = ArgentWebWallet.init({
appName: "hackbot",
Copy link
Owner

Choose a reason for hiding this comment

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

why here is hackbot? the project name is Spotnet


const argentWebWallet = ArgentWebWallet.init({
appName: "hackbot",
environment: "sepolia",
Copy link
Owner

Choose a reason for hiding this comment

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

we don't have contract in sepolia, only mainnet

setIsLoading(true);
try {
const call = {
contractAddress: ARGENT_DUMMY_CONTRACT_ADDRESS,
Copy link
Owner

Choose a reason for hiding this comment

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

why here dummy data if it should be user contract address?

const call = {
contractAddress: ARGENT_DUMMY_CONTRACT_ADDRESS,
entrypoint: ARGENT_DUMMY_CONTRACT_ENTRYPOINT,
calldata: ["0x1"],
Copy link
Owner

Choose a reason for hiding this comment

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

why here is dummy data if there should be input_data

@@ -14,6 +14,8 @@ export const kSTRK_ADDRESS = '0x045cd05ee2caaac3459b87e5e2480099d201be2f62243f83
export const ONE_HOUR_IN_MILLISECONDS = 3600000;

export const TELEGRAM_BOT_LINK = 'https://t.me/spotnet_bot';
// 'https://t.me/emarc99bot'
Copy link
Owner

Choose a reason for hiding this comment

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

Please, remove it when you push it to PR

@@ -8,6 +8,7 @@ import tailwindcss from '@tailwindcss/vite';
export default defineConfig({
server: {
port: 3000,
allowedHosts: ['c25f-98-97-78-151.ngrok-free.app'],
Copy link
Owner

Choose a reason for hiding this comment

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

Make it env variable

@djeck1432
Copy link
Owner

@emarc99 close this PR as I'm not sure, that you can finish this task on time

@djeck1432 djeck1432 closed this Feb 17, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants