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

docs: update docs #199

Merged
merged 6 commits into from
Sep 11, 2023
Merged

docs: update docs #199

merged 6 commits into from
Sep 11, 2023

Conversation

kingcre
Copy link
Contributor

@kingcre kingcre commented Sep 7, 2023

Description

This PR includes updates in API and CLI docs.

@kingcre kingcre self-assigned this Sep 7, 2023
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Sep 7, 2023
Copy link
Member

@crypin crypin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. Is there a reason why x/liquidamm/spec part is empty? Will it be added later?
  2. I think we need to update the module name used for this part
  3. This is another issue, but it seems necessary to discuss whether the voting power related to liquidamm balance including btoken should be added to the liquidstaking liquid gov part

@kingcre
Copy link
Contributor Author

kingcre commented Sep 11, 2023

@crypin

  1. Is there a reason why x/liquidamm/spec part is empty? Will it be added later?

Updated x/liquidamm/spec.

  1. I think we need to update the module name used for this part

Handled in #201, in this file.

  1. This is another issue, but it seems necessary to discuss whether the voting power related to liquidamm balance including btoken should be added to the liquidstaking liquid gov part

Handled in #201.

@kingcre kingcre requested a review from crypin September 11, 2023 01:51
A bidder only can place a single bid per auction of a liquid farm.
The bid amount of the pool coin must be higher than the current winning bid amount that is the highest bid amount of the auction at the moment.
The bidder placing the bid with the highest amount of the pool coin becomes the winner of the auction and will takes all the accumulated rewards amount at the end of the auction.
## LiquidAMM
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks still empty of some LiquidAmm parts

Is it missing or will it be added later in a separate PR?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should add it later in a separate PR.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If it's not going to proceed right away, it would be good to create an issue or task so that we don't miss it

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Opened #204

@crypin crypin self-requested a review September 11, 2023 08:47
@kingcre kingcre merged commit 325c028 into amm-module Sep 11, 2023
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation x/amm x/exchange x/liquidamm
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants