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.

Sign up for GitHub

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

Jump to bottom

feat: implement x/collection #574

Merged
merged 89 commits into from
Jul 19, 2022
Merged

feat: implement x/collection #574

merged 89 commits into from
Jul 19, 2022

Conversation

0Tech
Copy link
Collaborator

@0Tech 0Tech commented Jun 28, 2022 •
edited
Loading

Description

closes: #487

Checklist:

  • I followed the contributing guidelines and code of conduct.
  • I have added a relevant changelog to CHANGELOG.md
  • I have added tests to cover my changes.
  • I have updated the documentation accordingly.
  • I have updated API documentation client/docs/swagger-ui/swagger.yaml

0Tech added 30 commits May 30, 2022 07:48
@0Tech
feat: define x/collection proto
a1bd2c1
@0Tech
feat: update proto
e8ef9d1
@0Tech
feat: add event_spent and event_received
a5be418
@0Tech
feat: add x/collection to simapp
91b84af
@0Tech
feat: implement send
4772e08
@0Tech
feat: add getters to classes
934669a
@0Tech
feat: implement create
ef0e5a6
@0Tech
feat: add new msgs and deprecate the old ones
9a2001c
@0Tech
fix: fix the type of the event fields
9857ace
@0Tech
feat: add new queries and deprecate the old ones
e830346
@0Tech
feat: implement msgs
9cad6cf
@0Tech
fix: use coins
d37c105
@0Tech
feat: implement authorization logic
06fc5fc
@0Tech
feat: implement grant
e9ef079
@0Tech
feat: implement nft
cc357e9
@0Tech
feat: implement genesis
9278448
@0Tech
docs: update documents on proto
cd81839
@0Tech
docs: update version information in the comments
8260807
@0Tech
docs: update ERC777 related comments
c6ec4a0
@0Tech
test: add tests on genesis
7f360c4
@0Tech
feat: update protos
1a88643
@0Tech
fix: fix export genesis
4d86120
@0Tech
feat: return the id of the created entity
6f85fc2
@0Tech
feat: add next ids to genesis state
b94f926
@0Tech
feat: add queries
a34cc8f
@0Tech
fix: split EventMinted
97e62f8
@0Tech
feat: update field names and comments in tx proto
4d90df4
@0Tech
feat: update comments in query proto
fc7bd52
@0Tech
fix: fix event types and update proto
8a5990d
@0Tech
feat: add query client
c259200
zemyblue
zemyblue reviewed Jul 15, 2022
View reviewed changes
x/collection/keeper/grpc_query.go Outdated Show resolved Hide resolved
x/collection/keeper/grpc_query.go Outdated Show resolved Hide resolved
x/collection/keeper/grpc_query.go Outdated Show resolved Hide resolved
x/collection/keeper/keys.go Outdated Show resolved Hide resolved
x/collection/keeper/keys.go Outdated Show resolved Hide resolved
x/collection/keeper/nft.go Outdated Show resolved Hide resolved
zemyblue
zemyblue reviewed Jul 15, 2022
View reviewed changes
x/collection/keeper/nft.go Show resolved Hide resolved
x/collection/keeper/nft.go Show resolved Hide resolved
@0Tech 0Tech mentioned this pull request Jul 15, 2022
Merged
5 tasks
zemyblue
zemyblue reviewed Jul 18, 2022
View reviewed changes
x/collection/module/module.go Outdated Show resolved Hide resolved
x/collection/msgs.go Outdated Show resolved Hide resolved
x/collection/genesis.go Show resolved Hide resolved
@0Tech 0Tech requested review from zemyblue and dudong2 July 18, 2022 06:15
@0Tech 0Tech merged commit cb6ec96 into Finschia:main Jul 19, 2022
@0Tech 0Tech deleted the collection branch July 19, 2022 02:34
@zemyblue zemyblue mentioned this pull request Oct 27, 2022
Merged
5 tasks
@zemyblue zemyblue mentioned this pull request Nov 28, 2022
Merged
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zemyblue zemyblue zemyblue approved these changes

@dudong2 dudong2 dudong2 approved these changes

Assignees

@0Tech 0Tech

Labels
A: improvement Changes in existing functionality C:x/collection
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Migrate x/collection from line/link
3 participants
@0Tech @zemyblue @dudong2