Skip to content

Conversation

airinterface
Copy link
Collaborator

Pull Request Template

Description

This feature creates member Hat, and add owner to a member Hat at BigBang.

Fixes # issues/328

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Screenshots

If applicable, add screenshots to help explain your problem.

Copy link
Member

@yu23ki14 yu23ki14 left a comment

Choose a reason for hiding this comment

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

2点コメントいれました!

uint256 memberHatId = Hats.createHat(
hatterHatId,
defaultMemberHatName,
2,
Copy link
Member

Choose a reason for hiding this comment

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

Max supply を99個でお願いします

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

99個に変更しました。

// 3. Create Fixed Roles under TopHat
// 3. Create Member Hat ID
string memory defaultMemberHatName = "Member Hat for Organization";
string memory defaultMemberHatImage = _hatterHatImageURI;
Copy link
Member

Choose a reason for hiding this comment

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

こちらガス節約のために、変数定義せずそのまま関数に入れてもらって大丈夫です。

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@yu23ki14 さん、関数に導入しました。

@airinterface airinterface force-pushed the issues/328-MemberHatAtBigBang branch from 8d971d6 to 3af8bce Compare July 21, 2025 12:39
@yu23ki14 yu23ki14 changed the base branch from main to tobanv2 July 27, 2025 02:49
@yu23ki14 yu23ki14 merged commit 23c1a96 into tobanv2 Jul 27, 2025
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