Skip to content

Update HIP 1215 - #1295

Merged
mgarbs merged 4 commits into
hiero-ledger:mainfrom
stoyanov-st:hip-1215-update
Oct 13, 2025
Merged

Update HIP 1215#1295
mgarbs merged 4 commits into
hiero-ledger:mainfrom
stoyanov-st:hip-1215-update

Conversation

@stoyanov-st

Copy link
Copy Markdown
Contributor

Updating the HIP-1215 smart contract function selectors.

We want to provide more clarity to the users thus we change the sender wording to payer.
Also introduced is an item to the Open Issues section regarding an unsupported case for contract deployment.

Signed-off-by: Stanimir Stoyanov <stanimir.stoyanov@limechain.tech>
@stoyanov-st
stoyanov-st requested a review from a team as a code owner September 25, 2025 12:26
Signed-off-by: Stanimir Stoyanov <stanimir.stoyanov@limechain.tech>
Comment thread HIP/hip-1215.md Outdated
@gkozyryatskyy

Copy link
Copy Markdown
Contributor

One more thing. Is it make sense to add me to a working-group of the HIP?

Comment thread HIP/hip-1215.md
lukelee-sl
lukelee-sl previously approved these changes Oct 6, 2025

@lukelee-sl lukelee-sl 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.

LGTM

Signed-off-by: Stanimir Stoyanov <stanimir.stoyanov@limechain.tech>
Signed-off-by: Stanimir Stoyanov <stanimir.stoyanov@limechain.tech>

@lukelee-sl lukelee-sl 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.

LGTM

@gkozyryatskyy gkozyryatskyy 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.

LGTM! Thank you @stoyanov-st !

@tinker-michaelj tinker-michaelj 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.

LGTM, tyvm @stoyanov-st !

@mgarbs
mgarbs merged commit 53298f2 into hiero-ledger:main Oct 13, 2025
6 of 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.

5 participants