Skip to content

Added password prompt for CFund payment/proposal dialogs#474

Merged
mxaddict merged 1 commit intonavcoin:masterfrom
digitaloten:patch-2
May 14, 2019
Merged

Added password prompt for CFund payment/proposal dialogs#474
mxaddict merged 1 commit intonavcoin:masterfrom
digitaloten:patch-2

Conversation

@digitaloten
Copy link
Contributor

Added password prompt for CFund payment/proposal dialogs

This is for issue #457

@mxaddict
Copy link
Contributor

mxaddict commented May 9, 2019

utACK

@mxaddict
Copy link
Contributor

mxaddict commented May 9, 2019

Built on Ubuntu 18.04 (I'll do a testnet/devnet test for the functionality)

Copy link
Contributor

@mxaddict mxaddict left a comment

Choose a reason for hiding this comment

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

Tested on devnet, the prompts work for both proposal and payment requests.

@aguycalled
Copy link
Member

ACK tested on debian

@aguycalled
Copy link
Member

@francisjyap please leave a comment in the original issue with a navcoin address for payment

@mxaddict
Copy link
Contributor

mxaddict commented May 9, 2019

@aguycalled the test seems to keep failing (Timeout) Any ideas how to get the travis build to work?

@aguycalled
Copy link
Member

use travis_wait to extend the test job max execution time

@aguycalled
Copy link
Member

Looks like it is not possible to extend the job duration to more than 50 min for public repositories using the free version of travis. Any alternatives different from reducing our test units?

@mxaddict
Copy link
Contributor

mxaddict commented May 9, 2019

Looks like it is not possible to extend the job duration to more than 50 min for public repositories using the free version of travis. Any alternatives different from reducing our test units?

I think if we adjust testnet/devnet so the tests pass faster?

Like for example, we can have the static staking reward take only 150 (50 per step) blocks instead of the current 300 (100 per step)

And I think the same can be done for cfund tests as well

What do you think @aguycalled @craigmacgregor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants