Skip to content

Merge fixes for backport 7947 to stable2503#8269

Merged
acatangiu merged 8 commits intoparitytech:backport-7947-to-stable2503from
alistair-singh:alistair/backport-7947-to-stable2503
Apr 18, 2025
Merged

Merge fixes for backport 7947 to stable2503#8269
acatangiu merged 8 commits intoparitytech:backport-7947-to-stable2503from
alistair-singh:alistair/backport-7947-to-stable2503

Conversation

@alistair-singh
Copy link
Contributor

@alistair-singh alistair-singh commented Apr 16, 2025

Merge fixes for backport branch #8259

Had to cherry-pick the following commits:
#8020
#8038

claravanstaden and others added 4 commits April 17, 2025 01:26
- **Renames event in the Outbound Queue v2**: Resolves
[SNO-1430](https://linear.app/snowfork/issue/SNO-1430)
- **Remove unnecessary PaidFees event in Inbound Queue:** Resolves
[SNO-1437](https://linear.app/snowfork/issue/SNO-1437)
- **Remove unused types from Inbound Queue v2 pallet config:** Resolves
[SNO-1436](http://linear.app/snowfork/issue/SNO-1436)
- **Zero relayer reward in Inbound Queue should not be registered for a
reward:** Resolves
[SNO-1435](https://linear.app/snowfork/issue/SNO-1435)
- **Moves Snowbridge System Frontend Pallet index from `80` to `36` on
AH**:
paritytech#7402 (comment)
- **Add message rejected event to outbound queue:** Resolves
[SNO-1428](https://linear.app/snowfork/issue/SNO-1428) Closes
paritytech#7972
- **Removes unused method from Outbound Queue v2:** Resolves
[SNO-1431](https://linear.app/snowfork/issue/SNO-1431)
- **Fixes bug introduced in V2 related to commands on Ethereum's
indexes:** Introduced in
paritytech#7402, the command IDs
need to match on the [contract
side](https://github.com/Snowfork/snowbridge/blob/b2eb7a4e0e8f9ef6182ff378733453a9513cb76b/contracts/src/Types.sol#L78).
This ensures that when we upgrade BH with the new code, it will remain
compatible with the existing contracts on Ethereum that don't know
anything about V2.
- **Benchmarks `submit_delivery_receipt` in the Outbound Queue v2**
- **Allow transact without transfer of any asset**

---------

Co-authored-by: Ron <yrong1997@gmail.com>
…ch#8038)

For testing multi-hops transfer of PNA from Penpal to Ethereum, the
runtime configurations on Penpal have been revamped, including:

- Allow to teleport native asset from/to AH
- Allow to use native asset to pay fees
- Allow unpaid execution from relay chain for sudo calls

It's required for Snowfork/snowbridge#1419 to
setup a local test environment to transfer PNA from third-party
parachains. It demonstrates that a one-click tranfer from a Parachain to
Ethereum is possible. In this case, the first hop-from Penpal to AH, is
teleport-based, while the next hop-from AH to Ethereum, is
reserved-based.

---------

Co-authored-by: Adrian Catangiu <adrian@parity.io>
@paritytech-review-bot paritytech-review-bot bot requested a review from a team April 16, 2025 23:41
@alistair-singh alistair-singh changed the title Alistair/backport 7947 to stable2503 Merge fixes for backport 7947 to stable2503 Apr 16, 2025
@alistair-singh alistair-singh force-pushed the alistair/backport-7947-to-stable2503 branch from 33e003c to dc64a3f Compare April 17, 2025 00:22
@acatangiu
Copy link
Contributor

@alistair-singh can you not push these directly to backport-7947-to-stable2503 branch? The branch should not be protected, and review and all checks happen there before going to stable2503.

@acatangiu acatangiu merged commit 75bd46c into paritytech:backport-7947-to-stable2503 Apr 18, 2025
253 of 262 checks passed
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.

4 participants