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

Backport main changes (#900, #911, #912, #922, #923, #934, #939, #933, #934, #938) #937

Merged
merged 12 commits into from
Mar 22, 2023

Conversation

zemyblue
Copy link
Member

@zemyblue zemyblue commented Mar 17, 2023

Description

backporting list

Motivation and context

How has this been tested?

Screenshots (if appropriate):

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

tkxkd0159 and others added 8 commits March 17, 2023 17:48
* Fix: typos

Fix: typos

* Fix: typo

Fix: typo

* Fix: typo

Fix: typo

* go fmt

---------

Co-authored-by: Jayden Lee <41176085+tkxkd0159@users.noreply.github.com>
* Add uri to x/token

* Add tests

* Add uri to x/collection

* Add tests

* Update CHANGELOG.md

* Update CHANGELOG.md

* Emit both old and new change

* Add tests

* Remove xxx_uri from events

* Revert irrelevant changes

* Lint

(cherry picked from commit 81fec6d)
* feat: deprecate block mode in proto

* add changelog

(cherry picked from commit b626dec)
* Add censorship into proto

* Implement censorship

* Add FoundationExecProposal to proto

* Implement FoundationExecProposal

* Update documentation

* Update CHANGELOG.md

* Rename files

* Lint

* Add amino test on MsgUpdateCensorship

* Add pagination on cli

* Update x/foundation/README.md

* Update proposal cli description

* Chore

* Apply feedbacks

(cherry picked from commit d9428ec)
* Propagate events in x/foundation through sdk.Results

* Update CHANGELOG.md

(cherry picked from commit c025e91)
* Update links in documentation

* Update CHANGELOG.md

(cherry picked from commit 7623acc)
@zemyblue zemyblue changed the title Backport main changes Backport main changes (#900, #911, #912, #922, #923, #934) Mar 17, 2023
@zemyblue zemyblue self-assigned this Mar 17, 2023
@codecov
Copy link

codecov bot commented Mar 17, 2023

Codecov Report

Merging #937 (a6ca9e7) into rc/v0.47.0-alpha1 (11966d1) will increase coverage by 0.05%.
The diff coverage is 78.00%.

Additional details and impacted files

Impacted file tree graph

@@                  Coverage Diff                  @@
##           rc/v0.47.0-alpha1     #937      +/-   ##
=====================================================
+ Coverage              62.33%   62.38%   +0.05%     
=====================================================
  Files                    653      655       +2     
  Lines                  79467    79818     +351     
=====================================================
+ Hits                   49536    49795     +259     
- Misses                 27252    27338      +86     
- Partials                2679     2685       +6     
Impacted Files Coverage Δ
client/context.go 53.93% <0.00%> (ø)
crypto/keys/secp256k1/internal/secp256k1/curve.go 42.85% <ø> (ø)
simapp/test_helpers.go 9.96% <ø> (+0.10%) ⬆️
store/types/listening.go 68.42% <ø> (ø)
x/foundation/client/testutil/grpc.go 91.89% <ø> (-0.61%) ⬇️
x/foundation/keeper/internal/abci.go 75.00% <ø> (ø)
x/foundation/keeper/internal/grpc_query.go 3.90% <0.00%> (ø)
x/foundation/keeper/internal/invariants.go 100.00% <ø> (ø)
x/foundation/keeper/internal/keeper.go 89.47% <ø> (ø)
x/foundation/keeper/internal/migrations.go 0.00% <ø> (ø)
... and 32 more

... and 1 file with indirect coverage changes

dudong2
dudong2 previously approved these changes Mar 21, 2023
@zemyblue zemyblue changed the title Backport main changes (#900, #911, #912, #922, #923, #934) Backport main changes (#900, #911, #912, #922, #923, #934, #939) Mar 21, 2023
@zemyblue zemyblue requested a review from dudong2 March 21, 2023 07:33
loin3
loin3 previously approved these changes Mar 21, 2023
@0Tech
Copy link
Collaborator

0Tech commented Mar 21, 2023

Can you please add the latest PRs (#933, #934, #938)?

Also #939, for ostracon.

0Tech added 3 commits March 22, 2023 15:35
* Hide unnecessary apis

* Use x/foundation mocks

* Lint

* Update CHANGELOG.md

* Remove interface

* Revert "Remove interface"

This reverts commit 8dd124d.

(cherry picked from commit aabe516)
* Update links in documentation

* Update CHANGELOG.md

(cherry picked from commit 7623acc)
(cherry picked from commit b748d70)
* Add log into x/collection import

* Update CHANGELOG.md

(cherry picked from commit 7e85625)
@zemyblue zemyblue changed the title Backport main changes (#900, #911, #912, #922, #923, #934, #939) Backport main changes (#900, #911, #912, #922, #923, #934, #939, #933, #934, #938) Mar 22, 2023
@zemyblue zemyblue requested a review from loin3 March 22, 2023 07:04
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.

7 participants