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

fix: use connected pda seed for execute and execute spl #88

Merged
merged 1 commit into from
Mar 5, 2025

Conversation

skosito
Copy link
Contributor

@skosito skosito commented Mar 5, 2025

No need to have separate seed in connected program if executeSPL is called. That would mean if user uses 1 program to handle both, they would have to initialize 2 pda accounts with different seeds.

@skosito skosito requested a review from a team as a code owner March 5, 2025 04:29
@skosito skosito changed the title fix: use connected pda seed for execute spl as well fix: use connected pda seed for execute and execute spl Mar 5, 2025
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 0.74%. Comparing base (3296159) to head (6e64cc6).

Additional details and impacted files
@@          Coverage Diff          @@
##            main     #88   +/-   ##
=====================================
  Coverage   0.74%   0.74%           
=====================================
  Files          1       1           
  Lines        667     667           
=====================================
  Hits           5       5           
  Misses       662     662           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@skosito skosito merged commit 9316e78 into main Mar 5, 2025
15 checks passed
@skosito skosito deleted the fix-connected-pda-seed branch March 5, 2025 15:27
# 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.

4 participants