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: add cross chain txs for storage resources #50

Merged
merged 10 commits into from
Mar 17, 2023

Conversation

yutianwu
Copy link
Collaborator

@yutianwu yutianwu commented Feb 13, 2023

Description

This pull request will add transactions for mirroring resources from Greenfield to BSC, as well as operations for creating and deleting Buckets and Groups.

New messages will be added:

  • MirrorBucket
  • MirrorObject
  • MirrorGroup

New cross-chain apps will be implemented:

  • BucketApp: bucket app will handle the cross-chain functions like create bucket, delete bucket, mirror bucket
  • ObjectApp: object app will handle the cross-chain functions like delete object, mirror object
  • GroupApp: group app will handle the cross-chain functions like create group, delete group, mirror group

Rationale

To implement the cross-chain functions for the storage module.

Example

Mirror bucket:

gnfd tx storage mirror-bucket [bucket-id]

Mirror object:

gnfd tx storage mirror-object [object-id]

Mirror group:

gnfd tx storage mirror-group [group-id]

Changes

Notable changes:

  • add mirror txs for resources
  • implement cross chain apps for creating and deleting Buckets, Objects, and Groups

@yutianwu yutianwu force-pushed the add_mirror_storage branch 2 times, most recently from e89abe9 to 708a13c Compare February 15, 2023 12:23
x/storage/keeper/msg_server.go Outdated Show resolved Hide resolved
x/storage/keeper/msg_server.go Outdated Show resolved Hide resolved
@yutianwu yutianwu force-pushed the add_mirror_storage branch 2 times, most recently from 78f343f to 96c505d Compare February 20, 2023 06:23
@yutianwu yutianwu changed the title feat: add mirror txs for storage resources feat: add cross chain txs for storage resources Feb 20, 2023
@yutianwu yutianwu marked this pull request as ready for review February 20, 2023 07:49
@yutianwu yutianwu added the r4r label Feb 20, 2023
@yutianwu yutianwu force-pushed the add_mirror_storage branch 2 times, most recently from 2d82402 to 7c6c861 Compare February 22, 2023 07:00
Copy link
Collaborator

@unclezoro unclezoro left a comment

Choose a reason for hiding this comment

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

code reuse is the biggest problem, I believe you dont want to care about the handler logic change in the future.

@yutianwu yutianwu force-pushed the add_mirror_storage branch 4 times, most recently from 3a07607 to 9bcf681 Compare March 2, 2023 08:36
@yutianwu yutianwu force-pushed the add_mirror_storage branch 2 times, most recently from e5e2638 to 76f4a65 Compare March 6, 2023 08:31
@keefel
Copy link
Contributor

keefel commented Mar 14, 2023

Please fulfill the content in the PR template, with no deletion for the Rational and Examples part.

@yutianwu yutianwu force-pushed the add_mirror_storage branch 3 times, most recently from 0d77020 to 9c3ded9 Compare March 16, 2023 07:23
@yutianwu yutianwu force-pushed the add_mirror_storage branch from efe2fd5 to e8bba7a Compare March 17, 2023 02:06
proto/greenfield/storage/events.proto Outdated Show resolved Hide resolved
proto/greenfield/storage/events.proto Outdated Show resolved Hide resolved
x/storage/keeper/cross_app_bucket.go Show resolved Hide resolved
Copy link
Contributor

@fynnss fynnss left a comment

Choose a reason for hiding this comment

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

LGTM

@unclezoro unclezoro merged commit c5bcf4f into develop Mar 17, 2023
@unclezoro unclezoro deleted the add_mirror_storage branch April 18, 2023 09:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants