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: implement transfer_from and decrease_allowance #118

Conversation

chungquantin
Copy link
Collaborator

@chungquantin chungquantin commented Jul 23, 2024

Parent PR: #113
ISSUE: #104

Original wasm size: 30.7K, Optimized: 7.2K

Tasks

transfer_from

  • pallet-api implementation
  • pallet unit testing
  • contract integration test

decrease_allowance

  • pallet-api implementation
  • pallet unit testing
  • contract integration test
  • benchmarking

@chungquantin chungquantin changed the title Feature: implement transfer_from and decrease_allowance feat: implement transfer_from and decrease_allowance Jul 23, 2024
Copy link
Collaborator Author

@chungquantin chungquantin left a comment

Choose a reason for hiding this comment

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

Verify non-impacted code due to reformatting

@Daanvdplas
Copy link
Collaborator

Daanvdplas commented Jul 23, 2024

Could you please use the old formatting cargo +stable fmt for now? It creates a lot of noise in the PR and reviewers are already struggling with the size of the PR. We will change the formatting as soon as the api is merged in. As for your formatting for comments, that can stay of it is not automated.

@Daanvdplas Daanvdplas force-pushed the daan/feat-api_fungibles_pallet branch from 9d2276c to a8c4bb5 Compare July 23, 2024 17:28
pallets/api/src/fungibles/tests.rs Show resolved Hide resolved
pallets/api/src/fungibles/tests.rs Outdated Show resolved Hide resolved
pop-api/integration-tests/src/lib.rs Outdated Show resolved Hide resolved
pop-api/integration-tests/src/local_fungibles.rs Outdated Show resolved Hide resolved
pop-api/integration-tests/src/local_fungibles.rs Outdated Show resolved Hide resolved
pop-api/integration-tests/src/local_fungibles.rs Outdated Show resolved Hide resolved
pop-api/integration-tests/src/local_fungibles.rs Outdated Show resolved Hide resolved
pop-api/integration-tests/src/local_fungibles.rs Outdated Show resolved Hide resolved
@@ -367,6 +403,69 @@ fn transfer_works() {
});
}

#[test]
fn transfer_from_works() {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Could we add test situations for AssetNotLive. See other tests how to (freeze and destroy)

@chungquantin chungquantin requested a review from Daanvdplas July 24, 2024 09:36
@Daanvdplas
Copy link
Collaborator

This should be waited for review until #113 is approved

@Daanvdplas Daanvdplas deleted the branch r0gue-io:daan/feat-api_fungibles_pallet July 26, 2024 09:00
@Daanvdplas Daanvdplas closed this Jul 26, 2024
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.

2 participants