refactor: revert x/liquidfarming and add x/liquidamm #171
Conversation
the module was originally excluded from the simulation
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
| } | ||
| t, err := sdk.ParseTimeBytes(bz) | ||
| if err != nil { | ||
| panic(err) |
Check warning
Code scanning / CodeQL
Panic in BeginBock or EndBlock consensus methods Warning
| defer telemetry.ModuleMeasureSince(types.ModuleName, time.Now(), telemetry.MetricKeyBeginBlocker) | ||
|
|
||
| y, m, d := ctx.BlockTime().Date() | ||
| endTime, found := k.GetNextRewardsAuctionEndTime(ctx) |
Check warning
Code scanning / CodeQL
Panic in BeginBock or EndBlock consensus methods Warning
| nextEndTime = time.Date(y, m, d+1, 0, 0, 0, 0, time.UTC) // the next day 00:00 UTC | ||
| } | ||
| if err := k.AdvanceRewardsAuctions(ctx, nextEndTime); err != nil { | ||
| panic(err) |
Check warning
Code scanning / CodeQL
Panic in BeginBock or EndBlock consensus methods Warning
|
|
||
| // BeginBlock executes all ABCI BeginBlock logic respective to the module. | ||
| func (am AppModule) BeginBlock(ctx sdk.Context, _ abci.RequestBeginBlock) { | ||
| BeginBlocker(ctx, am.keeper) |
Check warning
Code scanning / CodeQL
Panic in BeginBock or EndBlock consensus methods Warning
There was a problem hiding this comment.
CodeQL found more than 10 potential problems in the proposed changes. Check the Files changed tab for more details.
kingcre
requested review from
crypin,
richard-bachman and
lv-koala
as code owners
Description
This PR reverts the modifications in the x/liquidfarming and creates a separate x/liquidamm module.
NOTE: merge this PR after merging #168
References