ProtoRev v16 Tracking Issue

[NotJeremyLiu]

Background

Tracking issue for all things related to ProtoRev for v16 upgrade

Concentrated Liquidity and ProtoRev Module Changes:

• Protorev: Use hooks instead of message parsing to trigger backruns #5045

Upgrades and ProtoRev Module Changes:

• Protorev: Update timing for pool updating and dev fee payout #4827

ProtoRev Module Changes:

• Refactor: integrate protorev into concentrated liquidity #4946

[p0mvn]

Hi @NotJeremyLiu . Is this up-to-date?

[NotJeremyLiu]

Hey @p0mvn ,

This is up to date.

I still need to turn the hook-based logic from a branch into a PR (need to add tests still), and the dev fee payout / updating is ready for review from Osmosis side (will add some comments on the PR to focus on a point that I would like extra eyes on.

[NotJeremyLiu]

Looks like I need to tackle some conflict issues now with #4827, can tackle tomorrow

[p0mvn]

Sweet. Thanks for the update

[NotJeremyLiu]

@p0mvn the two PRs in this issue are r4r now, added comments in the #4827 specifically would appreciate a focus on how the upgrade is handled, as I believe it is incorrect (e2e tests fail) but am at my limit of knowledge of how to fix it.

[p0mvn]

Thanks for the update @NotJeremyLiu . @stackman27 is looking into the e2e issue

[czarcas7ic]

Removing this tracking issue from sprint 32 since we are tracking the sub components in this sprint