Skip to content

Add Token.import_reissuable_token_script_pubkey#214

Merged
azuchi merged 2 commits into
masterfrom
issue_from_specified_pubkey
Jul 31, 2025
Merged

Add Token.import_reissuable_token_script_pubkey#214
azuchi merged 2 commits into
masterfrom
issue_from_specified_pubkey

Conversation

@rantan

@rantan rantan commented Jul 30, 2025

Copy link
Copy Markdown
Contributor

外部で発行された再発行可能トークンをインポートして扱うためのメソッドを追加しました。
追加したメソッドで、Glueby::Contract::AR::ReissuableTokenのレコードが作られることで、#reissue! が機能するようになります。

すでに #parse_from_payload で同様のことができますが、#to_payload で出力された値を前提とするため、glueby 以外で発行された再発行可能トークンのインポートとしては扱いにくいため、新しいインターフェースを追加します。

@rantan rantan force-pushed the issue_from_specified_pubkey branch from 3cfe185 to 0f57437 Compare July 31, 2025 02:08
@rantan rantan requested a review from azuchi July 31, 2025 05:48
Comment thread lib/glueby/contract/token.rb Outdated
Comment thread lib/glueby/contract/token.rb
Comment thread lib/glueby/contract/token.rb Outdated

@azuchi azuchi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

追加したメソッドで、Glueby::Contract::AR::ReissuableTokenのレコードが作られることで、#reissue! が機能するようになります。

この期待する動作について機能テストを追加できますか?

to import reissuable token from outside of Glueby
@rantan rantan force-pushed the issue_from_specified_pubkey branch from 0f57437 to fb8d7af Compare July 31, 2025 07:18
@rantan rantan changed the title Add Token.reissuable_token_from_script_pubkey Add Token.import_reissuable_token_script_pubkey Jul 31, 2025
@rantan

rantan commented Jul 31, 2025

Copy link
Copy Markdown
Contributor Author

追加したメソッドで、Glueby::Contract::AR::ReissuableTokenのレコードが作られることで、#reissue! が機能するようになります。

この期待する動作について機能テストを追加できますか?

機能テストを追加しました。

@azuchi azuchi merged commit 989b9bc into master Jul 31, 2025
8 checks passed
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