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.

Sign up for GitHub

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

Jump to bottom

feat: allow depositing vesting tokens in the vault #196

Merged
merged 30 commits into from
Aug 21, 2024
Merged

feat: allow depositing vesting tokens in the vault #196

merged 30 commits into from
Aug 21, 2024

Conversation

neitdung
Copy link
Collaborator

@neitdung neitdung commented Jul 14, 2024 •
edited
Loading

closes #90

Description

  • Modify bond and unbond function for allow user bond and unbond vesting token to vault contract by sending CustomMsg to mesh-security-provider module.

Note

  • Need to write new e2e test
  • Need to modify unit tests because ProviderCustomMsg doesn't work with current tests, I've created some mock.rs files for advoid this error.
  • Follow up PR on mesh-security

neitdung and others added 28 commits June 27, 2024 12:35
@neitdung
add config module addr
6649fb1
@neitdung
add sender param to bond func and modify tests
c967bf0
@neitdung
handle unbond message from sdk
6208794
@neitdung
revert adding config and func params
3764141
@neitdung
revert unused
0e37ad1
@neitdung
remove tab spacing
a9b50dd
@neitdung
add provider binding msg
3c31f0c
@neitdung
add dependencies
ae41e93
@neitdung
impl custom response for vault messgae
de9759a
@trinitys7
Add fake custom to all workspace using mesh-vault
00db35d
@neitdung
fix error
180d287
@neitdung
remove unused file
e723970
@neitdung
update dependencies
b50d126
@neitdung
remove unused config
d03c9cc
@neitdung
fix vault contract error
7d981de
@trinitys7
Add fake custom to all provider contract
a7c170f
@neitdung
Merge pull request #3 from decentrio/trinity/vesting-delegate
6b29335 
Fix: multitests
@trinitys7
Custom MtApp for multitests
28ae4cc
@neitdung
Fix message name for fitting workspace name
7a560ce
@neitdung
add vault mock contract
63dabbc
@neitdung
add mock for multitest
245169b
@trinitys7
Make the code compile
41f0f7d
@neitdung
remove function effect main logic
fc38e68
@neitdung
remove unused deps
5d83a84
@neitdung
remove fake custom
f30397b
@neitdung
remove unused code
942da1a
@neitdung
revert mock to origin contract
543850b
@neitdung
change custom message name
b34450a
@neitdung neitdung mentioned this pull request Jul 14, 2024
Closed
@neitdung neitdung marked this pull request as ready for review July 14, 2024 16:03
@neitdung neitdung mentioned this pull request Jul 14, 2024
Merged
@neitdung neitdung mentioned this pull request Jul 24, 2024
Merged
@JakeHartnell
Copy link
Collaborator

cargo fmt again. Maybe update your text editor to format on save. : )

@neitdung neitdung force-pushed the feat/vesting-delegate-remake branch from 9999841 to ad46461 Compare July 29, 2024 03:15
@neitdung neitdung mentioned this pull request Jul 29, 2024
Merged
@trinitys7 trinitys7 merged commit c6a34ff into osmosis-labs:main Aug 21, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Allow depositing vesting tokens in the vault
3 participants
@neitdung @JakeHartnell @trinitys7