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 the launch protocol for send - Closes #280 #550

Merged
merged 3 commits into from
Mar 13, 2018

Conversation

ginacontrino
Copy link
Contributor

What was the problem?

Prefilling the send form with an amount did not work yet

How did I fix it?

Made it work for the amount as well

How to test it?

While being logged in, try for example
http://localhost:8080/#/main/transactions?address=16313739661670634666L&amount=2

the send form should be filled after that

Review checklist

@ginacontrino ginacontrino self-assigned this Mar 13, 2018
@ginacontrino ginacontrino requested a review from slaweet March 13, 2018 12:37
@ginacontrino ginacontrino force-pushed the 280-send-launch-protocol branch 2 times, most recently from be61bbc to cea76f9 Compare March 13, 2018 14:46
@ginacontrino ginacontrino force-pushed the 280-send-launch-protocol branch from cea76f9 to 7235ec0 Compare March 13, 2018 14:50
@ginacontrino ginacontrino force-pushed the 280-send-launch-protocol branch from 5c9ad96 to 6b38fd6 Compare March 13, 2018 15:10
Copy link
Contributor

@slaweet slaweet left a comment

Choose a reason for hiding this comment

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

Thank you, Gina.

@ginacontrino ginacontrino merged commit b4d8019 into 0.3.0 Mar 13, 2018
@ginacontrino ginacontrino deleted the 280-send-launch-protocol branch March 13, 2018 16:08
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.

2 participants