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

Snowbridge - Disable create agent and create channel calls #506

Merged

Conversation

claravanstaden
Copy link
Contributor

Disable the Snowbridge create agent and channel calls, since they are not used and will be overhauled in v2.

@claravanstaden claravanstaden marked this pull request as ready for review December 9, 2024 08:22
@claravanstaden
Copy link
Contributor Author

@alistair-singh @vgeddes @yrong ready for review.

@acatangiu
Copy link
Contributor

@claravanstaden please fix clippy

@joepetrowski or @muharem this needs another rank2 review

# Conflicts:
#	relay/polkadot/src/coretime_migration.rs
@@ -226,6 +226,23 @@ parameter_types! {
pub const SS58Prefix: u8 = 0;
}

pub struct BaseFilter;
Copy link
Contributor

Choose a reason for hiding this comment

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

pub item should have docs.

@claravanstaden
Copy link
Contributor Author

/merge

@fellowship-merge-bot fellowship-merge-bot bot enabled auto-merge (squash) December 19, 2024 05:20
@fellowship-merge-bot
Copy link
Contributor

Enabled auto-merge in Pull Request

Available commands
  • /merge: Enables auto-merge for Pull Request
  • /merge cancel: Cancels auto-merge for Pull Request
  • /merge help: Shows this menu

For more information see the documentation

@fellowship-merge-bot fellowship-merge-bot bot merged commit 2e73a6c into polkadot-fellows:main Dec 19, 2024
48 checks passed
@claravanstaden claravanstaden deleted the disable-create-agent-channel branch December 19, 2024 07:41
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.

5 participants