Skip to content

Asem/v2 111/issue solana intent#27

Open
Asem-Abdelhady wants to merge 5 commits intolifinance:mainfrom
Asem-Abdelhady:asem/V2-111/issue-solana-intent
Open

Asem/v2 111/issue solana intent#27
Asem-Abdelhady wants to merge 5 commits intolifinance:mainfrom
Asem-Abdelhady:asem/V2-111/issue-solana-intent

Conversation

@Asem-Abdelhady
Copy link

This PR allows intents from EVM to Solana

important changes:

  • config.ts has solana data now
  • IssueIntent.svelte has 2 recipient fields(one optional for EVM) and (one required for Solana)

Copy link
Member

@reednaa reednaa left a comment

Choose a reason for hiding this comment

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

This looks good!!

I don't wanna approve since I don't want to merge changes into main:

  • Solana is not ready to be pushed to lintent.org.
  • I don't wanna break main build.

Also, I would love if we can fork off the develop branch which uses the @lifi/intent library.
For this PR, I believe it would be as easy as asking Claude fr it.

@reednaa
Copy link
Member

reednaa commented Mar 5, 2026

Also, can you make a branch on this repo?

@Asem-Abdelhady
Copy link
Author

This looks good!!

I don't wanna approve since I don't want to merge changes into main:

  • Solana is not ready to be pushed to lintent.org.
  • I don't wanna break main build.

Also, I would love if we can fork off the develop branch which uses the @lifi/intent library. For this PR, I believe it would be as easy as asking Claude fr it.

I am planning to make the PR to another branch most probbably will be "solana/develop". I will make "solana/develop" based on the current "develop" branch so i use the latest changes. Feel free to close this PR it is not going to be merged anyway.

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.

2 participants