Skip to content
This repository has been archived by the owner on Feb 26, 2021. It is now read-only.

reclaim tool: check states of all txs with proof address #516

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

chrisdukakis
Copy link
Contributor

No description provided.

@th0br0
Copy link
Contributor

th0br0 commented Nov 25, 2017

Could you explain the motivation behind this please?
We only need our current proof tx to confirm, not all tx for an address (which might never happen and neither is probably relevant)

@chrisdukakis
Copy link
Contributor Author

This actually checks if at least one tx with the proof address is confirmed, before enabling reveal option. We don't really care which one, and if it is a commitment or... reveal!
Problem with previous version is that we can't rely on ordering of txs as returned by findTrasnactionObjects(). Once a reveal tx is published, the process might stack right before reveal confirmation, if tool closes and reopens. It's half part of the issue you reported before initial release, and it really missed my full attention.

@th0br0
Copy link
Contributor

th0br0 commented Nov 26, 2017

We should only care about our own proof tx though. There could be multiple parties submitting proof transactions for a given address?

@chrisdukakis
Copy link
Contributor Author

Yes, exactly, anyone can repost it, but the proof is the address itself.

@jeremycraiglee
Copy link

Does anyone know when this fix will be included in a release?
I have encountered the same error while going through the reclaim process.

@Gecko100
Copy link

I started the reclaim tool in IOTA light wallet 2.5.4, used my old seed and put in a new seed. Hit go and it has been saying loading for almost 24 hours with a cog spinning... I need to shut my laptop down : . Does it normally take 24hours + ?

Will I lose my IOTA? can I restart it later?

@Alexzys
Copy link

Alexzys commented Dec 14, 2017

Hi, I have the same problem. What should I do?
Thank you very much

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants