Skip to content

Asset reissuance #107

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
altafan opened this issue Jun 16, 2020 · 4 comments · Fixed by #145
Closed

Asset reissuance #107

altafan opened this issue Jun 16, 2020 · 4 comments · Fixed by #145
Labels
story Viewpoint of the functionality

Comments

@altafan
Copy link
Collaborator

altafan commented Jun 16, 2020

This must be taken into account for adding support for creating reissuance transaction:

ElementsProject/elements#259

It will be mandatory to check that the input used for reissuing the asset (actually must be a token prev output) is confidential.

@altafan altafan added the story Viewpoint of the functionality label Jun 16, 2020
@altafan
Copy link
Collaborator Author

altafan commented Jun 17, 2020

Given the Elements issue above, and stated that similar to AddIssuance, this should be a function of the updater type, it will be necessary that the chosen partial input has a (Non)WitnessUtxo so that we can check that it's actually a confidential input, otherwise we should return an error.

This error should suggest the user move his reissuance token to a confidential address otherwise he will never be able to reissue his asset.

@altafan
Copy link
Collaborator Author

altafan commented Jun 24, 2020

It is mandatory that the input that contains the reissuance of the asset is the token output generated when this was issued.
Also, the token output must be added to the outputs of the reissuance transaction with exactly the same amount of the input.

Given tehese constrains, maybe it would be better that we provide a function that adds an input and a reissuance to it, and an output (the token output), rather than expecting them to be already present in the partial transaction and just add a reissuance?

cc @tiero

@tiero
Copy link
Member

tiero commented Jun 24, 2020

ACK

@tiero
Copy link
Member

tiero commented Jun 24, 2020

Could you split among the tickets you are working on already? @altafan

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
story Viewpoint of the functionality
Projects
None yet
2 participants