Skip to content
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

Fix not clearing input after sending via "send to this address" - Closes #847 #853

Merged
merged 2 commits into from May 30, 2018

Conversation

michaeltomasik
Copy link
Contributor

What was the problem?

Wallet: 'Send to address' field was still filled after sending via 'send to this address' in Explorer

How did I fix it?

Removed parameter from URL after sending transaction

How to test it?

Go to explorer, find 5201600508578320196L, click in 'send to this address', send any LSK amount, check if 'Send to address' contains 5201600508578320196Laddress

Review checklist

@michaeltomasik michaeltomasik self-assigned this May 29, 2018
@michaeltomasik michaeltomasik added this to New Issues in Sprint Board 21-05-18 via automation May 29, 2018
@michaeltomasik michaeltomasik added this to New Issues in Version 0.9.0 via automation May 29, 2018
@slaweet slaweet moved this from New Issues to Pull Requests in Sprint Board 21-05-18 May 30, 2018
@slaweet slaweet removed this from Pull Requests in Sprint Board 21-05-18 May 30, 2018
@slaweet slaweet moved this from New Issues to Pull Requests in Version 0.9.0 May 30, 2018
Copy link
Contributor

@yasharAyari yasharAyari left a comment

Choose a reason for hiding this comment

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

good job.

@michaeltomasik michaeltomasik merged commit 5e2c307 into 0.9.0 May 30, 2018
Version 0.9.0 automation moved this from Pull Requests to Merged Pull Requests May 30, 2018
@michaeltomasik michaeltomasik deleted the 847-not-clearning-input-after-send-in-explorer branch May 30, 2018 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Version 0.9.0
  
Merged Pull Requests
Development

Successfully merging this pull request may close these issues.

None yet

2 participants