DEVREL-468 feat: add oapp-solana - #1482
Conversation
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
Warning Review the following alerts detected in dependencies. According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.
|
d507c30 to
c1a40bc
Compare
14ee3ce to
dad23e4
Compare
St0rmBr3w
left a comment
There was a problem hiding this comment.
Given the time crunch I think this looks good. Some minor items, but we can improve rolling forward. Great work!
|
note: commits after 82a04bc do not change the program/contract code |
|
CI currently failing due to unrelated Ton test |
ItsAdel
left a comment
There was a problem hiding this comment.
I only ran the end to end tests. Approving based on that working. Did not get a chance to code review
Important: When merging into main, do not squash/rebase merge, since the audit will reference a commit hash in the feature branch. Only use merge commit approach.
Warning: do not use this in production, it is not yet audited. Although, in any case, even if you use this as a starting example for your production use case, you would still need to get your specific OApp implementation audited.
This PR will remain in draft mode until audit is completed.
One-liner
This PR adds a new example for a Solana OApp that interacts with an EVM OApp. The OApps pass strings to each other cross-chain.
Details
oappexample, with slight modifications (adding in enforced options support)Out of scope