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

Get rid of using seeds in VestingAddin instructions #19

Open
s-medvedev opened this issue Apr 1, 2022 · 0 comments
Open

Get rid of using seeds in VestingAddin instructions #19

s-medvedev opened this issue Apr 1, 2022 · 0 comments
Assignees

Comments

@s-medvedev
Copy link
Contributor

s-medvedev commented Apr 1, 2022

Problem: seeds-field used only to enable the creation of several vesting records for one owner. But it makes a problem for UI to find the correct seeds-value from known VestingRecord. To do this, you need to ask the user or look at the history to find a transaction to create.

Suggested solution: Use vesting_token_account as a seed to generate PDA for VestingRecord.

Rationale: This field already passed through instructions. Moreover, the internal implementation of vesting allows only a one-to-one link between VestingRecord and its vesting token account.
In this case, UI is available to find all VestingRecords using get_program_accounts method and then find the correct token_vesting_account.

┆Issue is synchronized with this Jira Task by Unito

@s-medvedev s-medvedev self-assigned this Apr 1, 2022
mich-master pushed a commit that referenced this issue Apr 1, 2022
Co-authored-by: Semen Medvedev <sm@neonlabs.org>
@sync-by-unito sync-by-unito bot closed this as completed Jun 14, 2022
@YYYurchenko YYYurchenko reopened this Jun 14, 2022
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

No branches or pull requests

2 participants