remove payment_id #26

Merged

wowario merged 1 commits from payment_id into master

wowario commented (Migrated from github.com)

Owner

removed payment_id option in wallet to discourage the use of unencrypted payment ids and added warning about their use.

🎉 1

mattcode55 (Migrated from github.com) reviewed

src/simplewallet/simplewallet.cpp

mattcode55 (Migrated from github.com) commented

Owner

Why is an integrated address recommended over no payment id?

wowario (Migrated from github.com) reviewed

src/simplewallet/simplewallet.cpp

wowario (Migrated from github.com) commented

Owner

Assuming a payment id is going to be used, an integrated address is recommended, but of course, no payment id at all is an option as well. I'll drop this sentence to avoid confusion and squash commits.

mattcode55 (Migrated from github.com) approved these changes

Reviewers

The pull request has been merged as 26bdc1e457.

Sign in to join this conversation.

No Label

No Milestone

No Assignees

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: wownero/wownero#26