Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Wallet names shouldn't use UUID #3481

Merged
merged 2 commits into from
Nov 18, 2016
Merged

Wallet names shouldn't use UUID #3481

merged 2 commits into from
Nov 18, 2016

Conversation

gavofyork
Copy link
Contributor

Fixes #3368

@gavofyork gavofyork added the A0-pleasereview 🤓 Pull request needs code review. label Nov 17, 2016
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.07%) to 85.929% when pulling 4fa361c on no-auto-uuid-name into 18f5703 on master.

@gavofyork gavofyork changed the title Wallet names shouldn't include uuid. Wallet names shouldn't use UUID Nov 17, 2016
@gavofyork gavofyork closed this Nov 17, 2016
@gavofyork gavofyork reopened this Nov 17, 2016
@gavofyork gavofyork added the M4-core ⛓ Core client code / Rust. label Nov 17, 2016
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.02%) to 85.981% when pulling 4fa361c on no-auto-uuid-name into 18f5703 on master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.007%) to 85.991% when pulling e02d4d9 on no-auto-uuid-name into 18f5703 on master.

@rphmeier rphmeier added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Nov 17, 2016
@gavofyork gavofyork merged commit b9add2f into master Nov 18, 2016
@gavofyork gavofyork deleted the no-auto-uuid-name branch November 18, 2016 11:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
A8-looksgood 🦄 Pull request is reviewed well. M4-core ⛓ Core client code / Rust.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@gavofyork @coveralls @rphmeier