Add override transfer to cli_wallet #560

Closed
opened 2023-09-26 12:51:37 +00:00 by christophersanborn · 6 comments
christophersanborn commented 2023-09-26 12:51:37 +00:00 (Migrated from gitlab.com)

As a user of cli_wallet, and an administrator (issuer) of one or more UIA tokens, I would like to be able to use cli_wallet to execute override_transfer operations for the management of my UIA token. As it stands currently, the cli_wallet does not offer a command to execute override transfer operations.

As a user of cli_wallet, and an administrator (issuer) of one or more UIA tokens, I would like to be able to use cli_wallet to execute override_transfer operations for the management of my UIA token. As it stands currently, the cli_wallet does not offer a command to execute override transfer operations.
christophersanborn commented 2023-09-26 12:57:28 +00:00 (Migrated from gitlab.com)

As it happens, @bgerze has already prepared a MR for this feature in a local fork, and I have given it initial review. We are ready to create a MR here, we just need to know what branch ( develop? beatrice ? ) we should base the MR off of.

@vampik — Could you advise on which branch to base the MR on?

Would be great to get this merged into the 1.6.0 mainnet release, but it could also be suited to a subsequent 1.6.1 point release.

As it happens, @bgerze has already prepared a MR for this feature in a local fork, and I have given it initial review. We are ready to create a MR here, we just need to know what branch ( `develop`? `beatrice` ? ) we should base the MR off of. @vampik — Could you advise on which branch to base the MR on? Would be great to get this merged into the 1.6.0 mainnet release, but it could also be suited to a subsequent 1.6.1 point release.
vampik commented 2023-09-26 13:09:37 +00:00 (Migrated from gitlab.com)

I think, it should be develop.
And then after we verify that everything works fine, we can proceed with merging to beatrice.

I think, it should be `develop`. And then after we verify that everything works fine, we can proceed with merging to `beatrice`.
bgerze commented 2023-09-26 18:48:17 +00:00 (Migrated from gitlab.com)

mentioned in merge request !247

mentioned in merge request !247
vampik commented 2023-11-07 06:23:08 +00:00 (Migrated from gitlab.com)

assigned to @christophersanborn, @vampik, and @bgerze

assigned to @christophersanborn, @vampik, and @bgerze
bgerze commented 2023-11-08 19:22:12 +00:00 (Migrated from gitlab.com)

mentioned in commit f679588274

mentioned in commit f679588274919bac936382026d4a6ce8a69a8f9a
vampik commented 2023-11-08 19:22:12 +00:00 (Migrated from gitlab.com)

mentioned in commit 726f9818f6

mentioned in commit 726f9818f62fa2bb79d20ad03181e33a91fe3525
vampik (Migrated from gitlab.com) closed this issue 2023-11-09 05:26:06 +00:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: Peerplays_Blockchain/peerplays_migrated#560
No description provided.