Skip to content

Conversation

@tWiLighT-xY91
Copy link

Description:

This PR moves examples/account_allowance_approve_transaction_hbar.py into the correct folder:

examples/account/account_allowance_approve_transaction_hbar.py

  • Updated changelog reference
  • Ensured the script runs with the new path
  • All changes scoped only to the issue requirements

Related issue(s):

Fixes #1004

Notes for reviewer:

N/A
Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

Signed-off-by: Shamitha Raj <shamitharaj318@gmail.com>
@exploreriii
Copy link
Contributor

Request review @tech0priyanshu

@github-actions
Copy link

github-actions bot commented Dec 8, 2025

Hi, this is WorkflowBot.
Your pull request cannot be merged as it is not passing all our workflow checks.
Please click on each check to review the logs and resolve issues so all checks pass.
To help you:

@@ -1,7 +1,5 @@
"""
Example demonstrating hbar allowance approval and usage.
Run:
Copy link
Contributor

Choose a reason for hiding this comment

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

please don't delete this line

Copy link
Contributor

Choose a reason for hiding this comment

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

this issue is outstanding, please take a look at what you are submitting:
https://github.com/hiero-ledger/hiero-sdk-python/pull/1007/files
ntoe you have deeted lines 1-4

Copy link
Contributor

@tech0priyanshu tech0priyanshu left a comment

Choose a reason for hiding this comment

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

The refactoring looks good - removing delete_hbar_allowance and negative tests from this file makes sense since they already exist in account_allowance_delete_transaction_hbar.py
Thanks, and welcome! 🎉 Great to have your first contribution much appreciated! @tWiLighT-xY91

@exploreriii exploreriii marked this pull request as draft December 9, 2025 18:17
@exploreriii
Copy link
Contributor

changing to draft while its ready to review again :)

@github-actions
Copy link

[commit-verification-bot]
Hi, this is VerificationBot.
Your pull request cannot be merged as it has unverified commits.
View your commit verification status: Commits Tab.

To achieve verified status, please read:

Remember, you require a GPG key and each commit must be signed with:
git commit -S -s -m "Your message here"

Thank you for contributing!

From the Hiero Python SDK Team

Signed-off-by: Shamitha Raj <shamitharaj318@gmail.com>
@github-actions
Copy link

Hi, this is MergeConflictBot.
Your pull request cannot be merged because it contains merge conflicts.

Please resolve these conflicts locally and push the changes.

To assist you, please read:

Thank you for contributing!

From the Hiero Python SDK Team

Signed-off-by: Shamitha Raj <twilightmorningstar32@gmail.com>
@tWiLighT-xY91 tWiLighT-xY91 marked this pull request as ready for review December 10, 2025 12:25
@@ -1,7 +1,5 @@
"""
Example demonstrating hbar allowance approval and usage.
Run:
Copy link
Contributor

Choose a reason for hiding this comment

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

this issue is outstanding, please take a look at what you are submitting:
https://github.com/hiero-ledger/hiero-sdk-python/pull/1007/files
ntoe you have deeted lines 1-4

@exploreriii exploreriii marked this pull request as draft December 10, 2025 15:26
@exploreriii
Copy link
Contributor

Hi @tWiLighT-xY91 do you need any help with this?

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.

[Good First Issue]: Move examples/account_allowance_approve_transaction_hbar.py

4 participants