Skip to content
This repository has been archived by the owner on Jun 11, 2024. It is now read-only.

Update forward of mainchain interoperability store #7623

Closed
Tracked by #7211
mitsuaki-u opened this issue Oct 7, 2022 · 1 comment
Closed
Tracked by #7211

Update forward of mainchain interoperability store #7623

mitsuaki-u opened this issue Oct 7, 2022 · 1 comment
Assignees
@mitsuaki-u
Labels
framework/module/interoperability Interoperability module
Milestone
Sprint 82

Comments

@mitsuaki-u
Copy link
Contributor

mitsuaki-u commented Oct 7, 2022 •
edited by ishantiw
Loading

Description

The forward function forwards a CCM to the specified receiving chain. This function is only called on the mainchain when processing a CCU. If an error occurs, the CCM is bounced back to the sending chain or discarded.

Should be updated according to updated LIP: 49.

Motivation

LIP 49 was updated and this function should reflect those changes.

Additional Information
@ishantiw ishantiw mentioned this issue Oct 10, 2022
Closed
71 tasks
@ishantiw ishantiw added the framework/module/interoperability Interoperability module label Oct 24, 2022
@shuse2 shuse2 added this to the Sprint 81 milestone Oct 24, 2022
@mitsuaki-u mitsuaki-u self-assigned this Nov 1, 2022
@sitetester
Copy link
Contributor

Related issue #7721

@shuse2 shuse2 modified the milestones: Sprint 81, Sprint 82 Nov 7, 2022
@shuse2 shuse2 mentioned this issue Nov 8, 2022
Merged
shuse2 added a commit that referenced this issue Nov 9, 2022
@shuse2
Update forward function for mainchain cross chain update command (#7754)
8ff7b14 
### What was the problem?

This PR resolves #7623 

### How was it solved?

- Add forward function in mainchain cross chain update command
- Refactor apply function to split verifyCCM function

### How was it tested?

- Add test cases for the new code
- Update existing tests
@shuse2 shuse2 closed this as completed Nov 9, 2022
ishantiw pushed a commit that referenced this issue Nov 14, 2022
@shuse2 @ishantiw
Update forward function for mainchain cross chain update command (#7754)
b035f85 
### What was the problem?

This PR resolves #7623 

### How was it solved?

- Add forward function in mainchain cross chain update command
- Refactor apply function to split verifyCCM function

### How was it tested?

- Add test cases for the new code
- Update existing tests
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@mitsuaki-u mitsuaki-u

Labels
framework/module/interoperability Interoperability module
Projects
None yet
Milestone
Sprint 82
Development

No branches or pull requests
4 participants
@sitetester @shuse2 @ishantiw @mitsuaki-u