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

EasyGas #242

Merged
merged 6 commits into from
Aug 19, 2023
Merged

EasyGas #242

merged 6 commits into from
Aug 19, 2023

Conversation

sanlee42
Copy link
Member

@sanlee42 sanlee42 commented Aug 1, 2023

Pull request type

Please check the type of change your PR introduces:

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Other information

@sanlee42 sanlee42 requested review from nkysg and WGB5445 August 1, 2023 02:23
@sanlee42 sanlee42 force-pushed the easygas-main branch 2 times, most recently from 29bad71 to dad3359 Compare August 1, 2023 10:04
CoreAddresses::assert_genesis_address(sender);
let genesis_account = GenesisSignerCapability::get_genesis_signer();
let gas_token_entry = GasTokenEntry { account_address, module_name, struct_name, data_source };
if (exists<GasTokenEntry>(address_of(&genesis_account))) {
Copy link
Contributor

Choose a reason for hiding this comment

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

这里是直接覆盖了?

Copy link
Member Author

Choose a reason for hiding this comment

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

是的,目前只支持STAR,未来可以升级成vec.

@nkysg nkysg requested review from jackzhhuang, welbon and simonjiao and removed request for WGB5445 August 15, 2023 12:36
@nkysg
Copy link
Contributor

nkysg commented Aug 16, 2023

@nkysg nkysg self-requested a review August 18, 2023 02:44
@nkysg nkysg merged commit 1164768 into main Aug 19, 2023
2 checks passed
@nkysg nkysg deleted the easygas-main branch August 19, 2023 02:31
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