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

Added amount parameter to farming pallet for deposit extrinsic #3581

Merged
merged 1 commit into from
May 10, 2023

Conversation

RustNinja
Copy link
Collaborator

@RustNinja RustNinja commented May 10, 2023

This PR introduced additional parameter amount to allow user deposit to specify any amount of LP tokens.
issue #3582

  • upgrade runtime
  • no node update

@kkast

@RustNinja RustNinja requested review from blasrodri and kkast May 10, 2023 10:03
@RustNinja RustNinja enabled auto-merge May 10, 2023 10:03
@RustNinja RustNinja added this pull request to the merge queue May 10, 2023
Merged via the queue into main with commit 7cd9235 May 10, 2023
@RustNinja RustNinja deleted the lp-staking-add-amount-parameter branch May 10, 2023 10:38
dzmitry-lahoda added a commit that referenced this pull request May 10, 2023
dzmitry-lahoda pushed a commit that referenced this pull request May 10, 2023
This PR introduced additional parameter `amount` to allow user `deposit`
to specify any amount of LP tokens.

- upgrade runtime
- no node update

@kkast
dzmitry-lahoda added a commit that referenced this pull request May 10, 2023
#3589)

This PR introduced additional parameter `amount` to allow user `deposit`
to specify any amount of LP tokens.

- upgrade runtime
- no node update

@kkast



- [ ] PR title is my best effort to provide summary of changes and has
clear text to be part of release notes
- [ ] I marked PR by `misc` label if it should not be in release notes
- [ ] I have linked Zenhub/Github or any other reference item if one
exists
- [ ] I was clear on what type of deployment required to release my
changes (node, runtime, contract, indexer, on chain operation, frontend,
infrastructure) if any in PR title or description
- [ ] I waited and did best effort for `Effect gate, automatically
merged if passed`(effects-job) to finish with success(green check mark)
with my changes
- [ ] I have added at least one reviewer in reviewers list
- [ ] I tagged(@) or used other form of notification of one person who I
think can handle best review of this PR

---------

Co-authored-by: Rust Ninja <[email protected]>
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