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

feat: enable arb filter for affiliate swap messages (backport #6890) #6894

Merged
merged 3 commits into from
Nov 20, 2023

Conversation

mattverse
Copy link
Member

Closes: #XXX

What is the purpose of the change

Manual Backport of @p0mvn work on affiliated swap messages PR https://github.com/osmosis-labs/osmosis/pull/6890/files

Testing and Verifying

This change added tests and can be verified as follows:

Documentation and Release Note

  • Does this pull request introduce a new feature or user-facing behavior changes?
  • Changelog entry added to Unreleased section of CHANGELOG.md?

Where is the change documented?

  • Specification (x/{module}/README.md)
  • Osmosis documentation site
  • Code comments?
  • N/A

@mattverse mattverse changed the base branch from main to v20.x November 20, 2023 01:47
@github-actions github-actions bot added C:x/lockup C:CLI C:x/gamm Changes, features and bugs related to the gamm module. C:x/incentives C:x/epochs T:build C:app-wiring Changes to the app folder C:simulator Edits simulator or simulations C:x/concentrated-liquidity labels Nov 20, 2023
Copy link
Member

@p0mvn p0mvn left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you

@p0mvn p0mvn closed this Nov 20, 2023
@p0mvn p0mvn reopened this Nov 20, 2023
@github-actions github-actions bot added C:x/txfees and removed C:x/lockup C:CLI C:x/gamm Changes, features and bugs related to the gamm module. C:x/incentives C:x/epochs T:build C:app-wiring Changes to the app folder C:simulator Edits simulator or simulations C:x/concentrated-liquidity labels Nov 20, 2023
@p0mvn
Copy link
Member

p0mvn commented Nov 20, 2023

Reopened for CI to run. Only 2 jobs were triggered

@p0mvn p0mvn merged commit 33db4e0 into v20.x Nov 20, 2023
1 check passed
@p0mvn p0mvn deleted the mattverse/roman-affiliated-swap branch November 20, 2023 02:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants